Skip to content

Conversation

@ujorgeleite
Copy link
Contributor

@ujorgeleite ujorgeleite commented Feb 21, 2022

Proposed changes (including videos or screenshots)

Implemented a sort filter for the marketplace screen. This component sorts the marketplace apps list in 4 ways, alphabetical order(A-Z), inverse alphabetical order(Z-A), most recently updated(MRU), and least recent updated(LRU). Besides that, I've generalized some components and types to increase code reusability, renamed some helpers as well as deleted some useless ones, and inserted the necessary new translations on the English i18n dictionary.
Demo gif:
Marketplace sort filter

Issue(s)

Steps to test or reproduce

Further comments

rique223 and others added 9 commits February 21, 2022 18:24
Generalized the FreePaidDropDownList into a component called RadioButtonList, started the implementation of the visual part of the marketplace sort filter component, created new sub folders for the Components folder of the Apps view and generalized the FreePaidDropDownPDefinitions type definition into a new type caled RadioDropDownDefinitions.
@ujorgeleite ujorgeleite force-pushed the feat-market-sort-filters branch from fed8d74 to 06c3b73 Compare February 21, 2022 21:33
@rique223 rique223 changed the title Translation hotfix [NEW] Marketplace sort filter Feb 21, 2022
@rique223 rique223 marked this pull request as ready for review February 22, 2022 00:04
@rique223 rique223 requested a review from a team February 22, 2022 00:05
@casalsgh casalsgh added this to the 4.5.0 milestone Feb 22, 2022
Copy link
Contributor

@geekgonecrazy geekgonecrazy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

From my perspective this looks good

@casalsgh casalsgh merged commit 7d823e2 into develop Feb 22, 2022
@casalsgh casalsgh deleted the feat-market-sort-filters branch February 22, 2022 23:46
@pierre-lehnen-rc pierre-lehnen-rc mentioned this pull request Mar 1, 2022
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