Repository navigation
feat(app-store): redesign the app store - #3534
Merged
Merged
Conversation
- show all tags with counts below the search, sorted by count and name, limited to one row with an additional tags drop for the overflow - add a grid/list view mode switch next to the search - add a subtitle and align the heading with the account pages - show authors and a raw download button at the bottom of the tiles - refactor the app store into smaller components, composables and helpers
- wider two column layout with header, gallery, description, information and versions - header with download button, required OpenCloud version, authors and tags - gallery with captions, counter and thumbnails instead of the cover image - information list with author, version, requirements and resources - compact version list showing three versions with a toggle for all of them - back link returns to the list including its filters - more horizontal padding in the app store
This reverts commit d07e73d.
Member
Author
|
Adjust no content message after #3556 got merged |
JammingBen
reviewed
Oct 6, 2026
JammingBen
left a comment
Member
There was a problem hiding this comment.
I noticed 2 issues when filtering via a tag:
- The number behind a tag is not accurate. It e.g. says "Viewer (5)" but then shows 6 apps. So there is some inconsistency in the matching.
- Filtering via a tag still shows the "All Apps" heading.
Member
Author
Yes, this is because clicking the tags did result in filling the search field and vice versa, I don't think we need this behaviour anymore, since we do now have the tag bar |
JammingBen
approved these changes
Oct 6, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
Redesigns the app store (list and details view) and modernizes its code.
List view
q_tagquery param, separate from the search, so the counts always match the shown appsofficialin the app json, authors and a download icon button at the bottomDetails view
Code
AppTagFilter,AppViewModeSwitch,AppDetailsHeader,AppDetailsGallery,AppDetailsInfo), composables (useAppFilter,useAppViewMode) and helpersdefineStoresetup for the apps store, named functions, no copy-pasted URL validation and route objectsScreenshots
Related Issue
How Has This Been Tested?
pnpm vite) against the docker compose backend, Chrome, widths 375–1600pxpnpm check:types,pnpm lint,pnpm format:checkTypes of changes