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.

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

Smiley list doesn't use correct sorting #5330

Closed
Gwenwyfar opened this issue Jan 6, 2019 · 4 comments · Fixed by #5342
Closed

Smiley list doesn't use correct sorting #5330

Gwenwyfar opened this issue Jan 6, 2019 · 4 comments · Fixed by #5342

Comments

@Gwenwyfar
Copy link
Contributor

The first sort type says "Default", but all it does is sort by the IDs. Clicking it does nothing.

@sbulen
Copy link
Contributor

sbulen commented Jan 6, 2019

All the sort columns sort by the content underneath. Clicking on "Default" sorts by the default flag, where set > not set, so sorting by "Default" pushes your default set to the bottom. So sorting by "default" is working.

What is a little different is that clicking it AGAIN doesn't do the reverse sort, like all the other sort buttons. I would expect a 2nd press to push the default set to the top.

EDIT: It would be a minor change to provide both ASC & DESC sorting to the Default column, which sorts by the "selected as default" flag. For some reason, it has never had that.

@sbulen
Copy link
Contributor

sbulen commented Jan 7, 2019

I had been testing on my WAMP server earlier. Oddly, on my Linux server, the sort is not correct... So there is something amiss here.

@Gwenwyfar
Copy link
Contributor Author

Gwenwyfar commented Jan 7, 2019

My default flag is in the middle. It does not change position either way. It is indeed a linux server.

@Sesquipedalian
Copy link
Member

Once #5342 is merged:

  1. The list of smiley sets will sort by "Name" by default, rather than by "Default" by default. (That's a weird sentence to type.)
  2. Sorting by "Default" actually will sort by whether the set is the default for the forum or not, rather than by ID.
  3. Sorting by ascending or descending will work as expected for the "Default" column.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants