Skip to content

fix: use outline svg icon for notifications - #2475

Merged
Antreesy merged 1 commit into
masterfrom
fix/noid/outline-icons-pt2
Aug 28, 2025
Merged

fix: use outline svg icon for notifications#2475
Antreesy merged 1 commit into
masterfrom
fix/noid/outline-icons-pt2

Conversation

@Antreesy

@Antreesy Antreesy commented Aug 28, 2025

Copy link
Copy Markdown
Collaborator

Signed-off-by: Maksim Sukharev <antreesy.web@gmail.com>

@nickvergessen nickvergessen 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.

But it's the app icon?

@Antreesy

Antreesy commented Aug 28, 2025

Copy link
Copy Markdown
Collaborator Author

But it's the app icon?

No, in header app is using mdi icon, not .svg asset. See screenshot 2
If you meant something different, then I'm not sure. Settings list should use the outline one, other places - up to discussion

@Antreesy
Antreesy merged commit c83b9d8 into master Aug 28, 2025
41 checks passed
@Antreesy
Antreesy deleted the fix/noid/outline-icons-pt2 branch August 28, 2025 16:16
@skjnldsv skjnldsv modified the milestones: Nextcloud 32, Nextcloud 33 Sep 28, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants