Skip to content

feat(ui): add highlights for notes and categories#1839

Open
aronovgj wants to merge 2 commits into
nextcloud:mainfrom
aronovgj:feature/highlights
Open

feat(ui): add highlights for notes and categories#1839
aronovgj wants to merge 2 commits into
nextcloud:mainfrom
aronovgj:feature/highlights

Conversation

@aronovgj
Copy link
Copy Markdown
Contributor

@aronovgj aronovgj commented May 1, 2026 •
edited
Loading

This PR updates Notes list/navigation behavior and styling to better match the Files app.

Changes:

  • Makes category selection highlights use the same blue active style as selected notes.
  • Updates active/drop-over category text and icons for proper contrast on blue.
  • Keeps All notes selected when opening a categorized note from the all-notes list.
  • Prevents automatic navigation jump to the note’s category while viewing All notes.

Extra:

  • Adjusts note list row spacing to match Files:
    • removes extra vertical note-row padding
    • adds a subtle 1px divider between notes

Fixes:
#1798

image

@github-actions @aronovgj
feat(ui): add highlights for notes and categories
877cd41 
Signed-off-by: Grigorij Aronov <aronovgj@gmx.net>
@aronovgj aronovgj mentioned this pull request May 1, 2026
Merged
enjeck
enjeck reviewed May 29, 2026
View reviewed changes
Copy link
Copy Markdown
Contributor

@enjeck enjeck left a comment

Choose a reason for hiding this comment

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

please check merge conflicts

@aronovgj
Merge remote-tracking branch 'upstream/main' into feature/highlights
f00c33b 
Signed-off-by: Grigorij Aronov <aronovgj@gmx.net>

# Conflicts:
#	composer.lock
#	src/components/CategoriesList.vue
@aronovgj aronovgj requested a review from enjeck May 29, 2026 16:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@silverkszlo silverkszlo Awaiting requested review from silverkszlo silverkszlo is a code owner

@enjeck enjeck Awaiting requested review from enjeck enjeck is a code owner

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@aronovgj @enjeck