Right now, the sort field is just part of the larger project navigation form, and is hard-coded.

We should:
1. Rename the sorts to be the same as d.o expects rather than being inconsistent.
2. Clean up the sort field handling so that we provide all available sorts.
3. Separate out the sort field so that it can be used by more than 1 form.

CommentFileSizeAuthor
#2 888400-2.project_solr.patch4.42 KBcsevb10
#1 888400-2.project_solr.patch4.43 KBcsevb10

Comments

csevb10’s picture

csevb10 commented
Status: Active » Needs review
StatusFileSize
new 888400-2.project_solr.patch4.43 KB

Patch to address the above.

csevb10’s picture

csevb10 commented
StatusFileSize
new 888400-2.project_solr.patch4.42 KB

Just a comment change.

csevb10’s picture

csevb10 commented
Issue tags: +drupal.org redesign, +drupal.org redesign solr, +drupal.org redesign project, +drupal.org redesign sprint 2
csevb10’s picture

csevb10 commented

The follow-up to this is to add the sort form in drupalorg for use on the search page: #888478: Add sort to the default search page

dww’s picture

dww
we/he/they
commented
Status: Needs review » Fixed

Looks great. Committed to HEAD, merged into both d.o and redesign.d.o bzr branches, and deployed. Thanks!

Status: Fixed » Closed (fixed)
Issue tags: -drupal.org redesign, -drupal.org redesign solr, -drupal.org redesign project, -drupal.org redesign sprint 2

Automatically closed -- issue fixed for 2 weeks with no activity.

Issue tags: +drupal.org redesign, +drupal.org redesign solr, +drupal.org redesign project, +drupal.org redesign sprint 2

Restoring issue tags, see #2125755: System messages removed all issue tags during D7 upgrade.