Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Create button "Sort by ..." #290

Closed
plutonik-a opened this issue Nov 13, 2017 · 3 comments
Closed

Create button "Sort by ..." #290

plutonik-a opened this issue Nov 13, 2017 · 3 comments
Assignees
@tuurma @plutonik-a
Milestone
2.1 FRUS Advanced...

Comments

@plutonik-a
Copy link
Contributor

Text next to the button is:
"Sort Results By"

Sort options for the results in the dropdown button are:

  1. Relevance - default
  2. Dates (oldest first)
  3. Dates (most recent first)

bildschirmfoto 2017-11-13 um 13 50 30

Parent: #284
@plutonik-a plutonik-a added this to the 2.1 FRUS Advanced search milestone Nov 13, 2017
@plutonik-a plutonik-a mentioned this issue Nov 13, 2017
Open
11 tasks
@plutonik-a plutonik-a self-assigned this Nov 28, 2017
plutonik-a added a commit that referenced this issue Nov 28, 2017
@plutonik-a
feat(search): Serialize "sort-by" options #290
3000ad7
plutonik-a added a commit that referenced this issue Nov 28, 2017
@plutonik-a
feat(search): Create "sort-by" template #290
3a946b7 
* refactor former template _filter-querystring.html together with new
include _sort-by-button.html into a wrapping template _filter-row.html,
because both includes needed to be placed into the same row
* rename inludes in calling page templates
plutonik-a added a commit that referenced this issue Nov 28, 2017
@plutonik-a
feat(search): Apply styling to "sort-by" button #290
ea8d4aa
plutonik-a added a commit that referenced this issue Nov 28, 2017
@plutonik-a
feat(search): Parameterize "sort-by" options #290
3265e4e 
TODO: Switching values still needs fixing, because only the default
case is working.
@plutonik-a
Copy link
Contributor Author

Status:
UI is ready, parameters needs further fixes for serialization.

bildschirmfoto 2017-11-28 um 16 15 30

bildschirmfoto 2017-11-28 um 16 15 41

plutonik-a added a commit that referenced this issue Dec 1, 2017
@plutonik-a
fix(search): Move sorting serialization into the right order #290
2c078ad
@plutonik-a plutonik-a assigned tuurma and plutonik-a and unassigned plutonik-a Dec 4, 2017
@joewiz
Copy link
Member

joewiz commented Dec 5, 2017

Back end is now prepared for this control to become active, with c929b55 and 25b3ee2.

Compare:

@tuurma
Copy link
Contributor

tuurma commented Dec 5, 2017

completed with a08041a and c8f39d9

@tuurma tuurma closed this as completed Dec 5, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@tuurma tuurma

@plutonik-a plutonik-a

Labels
None yet
Projects
None yet
Milestone
2.1 FRUS Advanced search
Development

No branches or pull requests
3 participants
@joewiz @tuurma @plutonik-a