Problem/Motivation

It isn't clear how the results are currently sorted, nor is there a way to change how they are sorted.

Proposed resolution

State how the results are being sorted, along with an ability to switch the sort criteria. Options could include:
- Most installed
- Most starred
- Date of most recent release

Remaining tasks

Decide on which sort criteria to use on each of the "recommended" and "all" search/browser pages.

User interface changes

Adding exposed sort dropdown.

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

Comments

mandclu created an issue. See original summary.

hmendes’s picture

chrisfromredfin’s picture

Yes, I think this will be required for MVP. For MVP sort, we (Leslie and I) believe the two options should be (1) number of downloads (or active installs) and (2) name (alphabetical by project name).

Open to discussion if something else adds value.

mandclu’s picture

I would say recency of release should be another sort, but maybe that doesn't have to be MVP.

I would encourage us to treat filtering to projects with a stable release as a priority also.

Finally, based on my (admittedly limited) use of the current state, it doesn't appear to filter results to projects compatible with the current site (version of Drupal etc). I realize that's likely off topic for a ticket about sort options, but I definitely think that should be MVP. Happy to create a new ticket for that if there isn't one already.

chrisfromredfin’s picture

Leslie and I are about to file a BUNCH of issues, and one is how to algorithmically determine "recommended," so that will be a good place for that other discussion. We did discuss "recency of release" but decided that might make more sense only on the "All" set of filters, but are open to discussion on it.

mandclu’s picture

StatusFileSize
new90.72 KB

"Last release" is a sort currently available for the search on drupal.org, so hopefully it shouldn't be a big lift to include that. I know a lot of devs who consider this an important way to evaluate modules.

chrisfromredfin’s picture

Issue summary: View changes
mandclu’s picture

Added #3240321: Only show projects compatible with the current site as a child issue to capture a requirement to only show projects compatible with the site's current major version of Drupal, which I think should really be included for MVP.

chrisfromredfin’s picture

After thinking about this more today, I think we need an exposed sort with the following options:
* relevance (since you'll be doing keyword search; and this would be the default if you keyword-search)
* popularity ("most installed" in Drupal.org parlance)
* alphabetical (in case someone is searching for something specific? or maybe not needed at all? currently "title" on d.o)

We could, to this, possibly add:
* recent activity (kind of like "last release" - or could just be last release)

bnjmnm made their first commit to this issue’s fork.

bnjmnm’s picture

Status: Active » Needs review
StatusFileSize
new94.71 KB

The MR makes it possible to sort by the following properties:

These are the properties that can easily be sorted by after the changes implemented in #3249958: Mirror API in database to make sorting and other queries possible. That issue removed the need to depend on D.O. api functionality, and it's all done with local database queries.

I think it could be useful to commit this one -- so sorting is actually possible -- and open followups for additional sort properties. Additional sort properties would likely require expanding the hook_install process to compute additional properties. That isn't difficult, but I think there's benefit of getting this level of sorting committed asap, and we can expand as needed.

tim.plunkett made their first commit to this issue’s fork.

tim.plunkett’s picture

Status: Needs review » Fixed

Merged! Thanks Ben

tim.plunkett’s picture

Status: Fixed » Reviewed & tested by the community

Uhh idk why it says that I committed stuff. I just tried to use the merge button...

tim.plunkett’s picture

Status: Reviewed & tested by the community » Fixed

Tried again and it seems like it worked

Status: Fixed » Closed (fixed)

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