Problem/Motivation

When viewing the UI Examples overview there is currently no sorting, we should make sure that the examples get sorted based on their id property.

Steps to reproduce

Proposed resolution

Remaining tasks

User interface changes

API changes

Data model changes

Issue fork ui_examples-3574277

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:

About issue forks

Comments

yannickoo created an issue. See original summary.

yannickoo’s picture

yannickoo
Location Berlin
commented
Assigned: Unassigned » yannickoo

yannickoo opened merge request !26

yannickoo’s picture

yannickoo
Location Berlin
commented
Assigned: yannickoo » Unassigned
Status: Active » Needs review
grimreaper’s picture

grimreaper
Primary language French
Location France 🇫🇷
commented
Assigned: Unassigned » grimreaper
grimreaper’s picture

grimreaper
Primary language French
Location France 🇫🇷
commented
Title: Order UI examples alphabetically based on their ID property » Order UI examples with the same logic as the other UI Suite modules
Assigned: grimreaper » Unassigned
Status: Needs review » Needs work
Issue tags: +Needs tests

Thanks @yannickoo for the issue and MR.

I put a comment in the MR, also needs tests.

yannickoo’s picture

yannickoo
Location Berlin
commented
Status: Needs work » Needs review

I think we can close this since the sorting has been implemented when I was introducing the new category property to the example definition in #3574276: Implement "category" property to organize UI Examples better.

yannickoo closed merge request !26

yannickoo’s picture

yannickoo
Location Berlin
commented
Status: Needs review » Closed (outdated)

Now that this issue is closed, review the contribution record.

As a contributor, attribute any organization that helped you, or if you volunteered your own time.

Maintainers, credit people who helped resolve this issue.

grimreaper’s picture

grimreaper
Primary language French
Location France 🇫🇷
commented
Issue tags: -Needs tests

Ok, let's do it in a single issue :)