Skip to content

App menu visual polishing - #62963

Merged
susnux merged 3 commits into
masterfrom
feat/app-menu-default-order
Aug 11, 2026
Merged

App menu visual polishing#62963
susnux merged 3 commits into
masterfrom
feat/app-menu-default-order

Conversation

@jancborchardt

@jancborchardt jancborchardt commented Aug 6, 2026

Copy link
Copy Markdown
Member

Summary

Sorting split off into #63104 as requested by @susnux at #62963 (comment)

Some menu polishing:

  • Subdued the "More apps" (for admins) / "App store" (for users) entry, especially since the main app icons became less intense with fix(core): improve app icon contrast in the app menu #62931 – but also its name is bolded when active, just like the other apps
  • Removed the "Appstore" entry which duplicated "More apps" for admins. As a result, the logic which provided the header title and icon needed to be reworked. Bonus: "Help & privacy" now also has proper icon and title instead of cog + "Settings".
Before After
Screenshot From 2026-08-06 13-38-50 Screenshot From 2026-08-06 14-00-31

Checklist

AI (if applicable)

  • The content of this PR was partly or fully generated using AI

@jancborchardt jancborchardt added this to the Nextcloud 35 milestone Aug 6, 2026
@jancborchardt jancborchardt self-assigned this Aug 6, 2026
@jancborchardt
jancborchardt requested review from a team as code owners August 6, 2026 12:08
@jancborchardt
jancborchardt requested review from skjnldsv and sorbaugh and removed request for a team August 6, 2026 12:08
@jancborchardt jancborchardt added the design Design, UI, UX, etc. label Aug 6, 2026
@jancborchardt jancborchardt added 3. to review Waiting for reviews papercut Annoying recurring UX issue with possibly simple fix. feature: apps management AI assisted labels Aug 6, 2026
@jancborchardt
jancborchardt requested review from leftybournes, provokateurin and salmart-dev and removed request for a team August 6, 2026 12:08

@kra-mo kra-mo left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Looks good from the screenshots/description.

But the app order should probably also be updated downstream to match, no?

@jancborchardt

jancborchardt commented Aug 6, 2026

Copy link
Copy Markdown
Member Author

But the app order should probably also be updated downstream to match, no?

Just in my opinion: The concept of app order defined by apps themselves does not seem to be a good solution – otherwise everyone would just fight for -999. We should just go for hardcoded Hub app order like here, and after that we could go alphabetical or smth of the sort.

@szaimen

szaimen commented Aug 6, 2026

Copy link
Copy Markdown
Contributor

Question: wouldnt it be even possible to add some headings into the menu to actuallly separate the apps based on their category inside this menu?

@jancborchardt

Copy link
Copy Markdown
Member Author

Question: wouldnt it be even possible to add some headings into the menu to actuallly separate the apps based on their category inside this menu?

Certainly would be possible and could be considered if Hub grows more, but I think that’s too much at least for now.

@jancborchardt

Copy link
Copy Markdown
Member Author

/compile

artonge
artonge previously requested changes Aug 6, 2026
Comment thread lib/private/NavigationManager.php Outdated
@jancborchardt
jancborchardt force-pushed the feat/app-menu-default-order branch from 5f04ab1 to 1354275 Compare August 6, 2026 12:56
@jancborchardt

Copy link
Copy Markdown
Member Author

Rector fails on master too, so unrelated

@jancborchardt
jancborchardt requested a review from artonge August 6, 2026 13:13
Comment thread apps/appstore/appinfo/info.xml
Comment thread core/src/components/AppMenu.vue Outdated
Comment thread lib/private/NavigationManager.php Outdated
@jancborchardt
jancborchardt force-pushed the feat/app-menu-default-order branch from 1354275 to 965290e Compare August 6, 2026 21:35
Comment thread core/src/components/AppMenu.vue Outdated
Comment thread core/src/components/AppMenu.vue Outdated
Comment thread core/src/components/AppMenu.vue Outdated
@jancborchardt
jancborchardt force-pushed the feat/app-menu-default-order branch 2 times, most recently from 1366f15 to b4b4c55 Compare August 10, 2026 08:47
@jancborchardt

Copy link
Copy Markdown
Member Author

/compile

@jancborchardt

Copy link
Copy Markdown
Member Author

@CarlSchwan yeah, removed those extra comments. :)

@jancborchardt jancborchardt moved this to 🏗️ At engineering in 🖍 Design team Aug 10, 2026
@jancborchardt jancborchardt mentioned this pull request Aug 10, 2026
9 tasks
@jancborchardt jancborchardt changed the title App menu: Fix default order and related issues App menu visual polishing Aug 10, 2026
@jancborchardt

Copy link
Copy Markdown
Member Author

And split out the sorting as per @susnux recommendation to #63104 :)

@jancborchardt
jancborchardt force-pushed the feat/app-menu-default-order branch from 2ae880c to 4325418 Compare August 10, 2026 10:48
jancborchardt and others added 3 commits August 11, 2026 01:13
Assisted-by: ClaudeCode:claude-opus-5
Signed-off-by: Jan C. Borchardt <925062+jancborchardt@users.noreply.github.com>
Assisted-by: ClaudeCode:claude-opus-5
Signed-off-by: Jan C. Borchardt <925062+jancborchardt@users.noreply.github.com>
Signed-off-by: nextcloud-command <nextcloud-command@users.noreply.github.com>
@jancborchardt
jancborchardt force-pushed the feat/app-menu-default-order branch from 4325418 to 21c5152 Compare August 10, 2026 23:16
@susnux
susnux merged commit c849c3b into master Aug 11, 2026
199 checks passed
@susnux
susnux deleted the feat/app-menu-default-order branch August 11, 2026 08:18
@github-project-automation github-project-automation Bot moved this from 🏗️ At engineering to 🎉 Done in 🖍 Design team Aug 11, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to review Waiting for reviews AI assisted design Design, UI, UX, etc. enhancement feature: apps management papercut Annoying recurring UX issue with possibly simple fix.

Projects

Status: 🎉 Done

Development

Successfully merging this pull request may close these issues.

8 participants