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

Intermediate alphabetical sorting #301

Merged
merged 6 commits into from
Mar 26, 2021
Merged

Intermediate alphabetical sorting #301

merged 6 commits into from
Mar 26, 2021

Conversation

triplingual
Copy link
Collaborator

@triplingual triplingual commented May 1, 2020

I was having trouble reading the list and propose sorting sections alphabetically, case-insensitively, that are not otherwise ordered intentionally. I recognise that #149 talks about intentional ordering in all sections. This PR makes the list more readable, but may conflict with the discussion there.

I sorted alphabetically (Latin), case-insensitively the sections
that seemed evidently sortable, omitting clearly non-sorted ones
(e.g. TOC) and ones that I am more confident are intentionally
ordered (e.g. Standards).
@triplingual triplingual changed the title Intermediate Intermediate alphabetical sorting May 1, 2020
@julsraemy
Copy link
Collaborator

Seems like a good idea, what do you think @regisrob, @jronallo ?

@regisrob
Copy link
Member

regisrob commented May 4, 2020

It is indeed the most neutral way of organizing lists. It conflicts with what is proposed in #149, even if I think it would be quite delicate to curate the list this way (i.e by making recommandations on the "preferred" items in each sub-list): in my opinion it is too subjective, too dependent on the user's standpoint... Btw it is not applicable to all sections, and it also makes the awesome-iiif list harder to curate and maintain.
So I think @triplingual's proposal does not hurt.
If we do that, we should consider updating the contribution guidelines (which states that new items should be added to the end of the list).

@jronallo
Copy link
Collaborator

I was initially opposed to alphabetical ordering as I feel like part of the task is to provide some recommendations through ordering. But if others like the alphabetical ordering I can be swayed by the argument about saving time in managing the list.

While it hadn't been followed all of the time, generally the most recent entries were added to the end of any sub-list. That doesn't get to recommendations, but does give some sense of how long something has been around which can factor into decisions. But even then that can be independently verified and considered (or not) in anyone's own evaluative rubric for making decisions about what to try. And the community is available for recommendations as well.

If we go alphabetical, I'd like to see a note at the top of the list stating that is the ordering principle and, if it would be alright, also state that ordering is in no way indicative of a recommendation. The guidelines for pull requests should also be updated at the same time as this pull request is merged (or shortly after) to ask that new entries be inserted in alphabetical order by name.

@triplingual
Copy link
Collaborator Author

The recommendation bit is indeed tricky. Is there a way the official list maintainers could agree on an icon to indicate recommended status? Could be put at the top of the list, e.g. “✅ = recommended”. With that, order will matter a little less. It’s going to get non-alphabetical sooner or later, whether through neglect or accident, so I feel like some sort of recommendation icon would be useful whatever the outcome of this PR.

Having the last item be the most recent gives a relative duration indicator, but not a terribly meaningful one IMO. Bottom is bottom, whether the preceding ones were added 5 minutes or 5 months ago. If there were a way to automate adding a Date Added stamp, ordering by recency could be handy. Otherwise, as a consumer of this list I found the existing order to be a mild hindrance.

@hadro
Copy link
Contributor

hadro commented Mar 19, 2021

I've opened up #335 to continue the discussion around highlights/recommendation etc., but I'd like to propose we go ahead with @triplingual's PR here to reorder the sub-lists by alpha sort and not let that block the other discussion, given that the way things are being contributed to the list now is not necessarily following any best practice.

If we go alphabetical, I'd like to see a note at the top of the list stating that is the ordering principle and, if it would be alright, also state that ordering is in no way indicative of a recommendation. The guidelines for pull requests should also be updated at the same time as this pull request is merged (or shortly after) to ask that new entries be inserted in alphabetical order by name.

I also agree with @jronallo's comment here, so I will see if I can do a quick addition to address these elements.

@triplingual
Copy link
Collaborator Author

In the interim, I'll update this from the upstream IIIF repo to make sure I have additions since I originally put this in.

NB: Did not alphabetize Contents (duh), Standards, or Community. I'm assuming the latter two are in an intentional order.
@hadro
Copy link
Contributor

hadro commented Mar 19, 2021

Thanks, Trip! I'm not good enough at git to figure out how to merge this into your branch, but here's a suggested updated bit of language for the disclaimer:

Disclaimer: The lists below are presented in alphabetical order within each section, and do not suggest any recommendation. Likewise, this list is created for informational purposes only and any links do not constitute an endorsement, recommendation, or favoring by the IIIF Consortium.

@hadro hadro merged commit fa3db2e into IIIF:master Mar 26, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants