Skip to content

Conversation

@susnux
Copy link
Contributor

@susnux susnux commented Jul 29, 2025

Summary

app icons in the header navigation are excluded from the outlined icons change, so this reverts this change.

Checklist

@susnux susnux added this to the Nextcloud 32 milestone Jul 29, 2025
@susnux susnux requested a review from a team as a code owner July 29, 2025 15:06
@susnux susnux removed the request for review from a team July 29, 2025 15:06
@susnux susnux added the bug label Jul 29, 2025
@susnux susnux requested review from artonge, nfebe and szaimen July 29, 2025 15:06
@susnux susnux added design Design, UI, UX, etc. 3. to review Waiting for reviews labels Jul 29, 2025
@szaimen szaimen requested review from JuliaKirschenheuter and removed request for szaimen July 29, 2025 15:24
@susnux susnux requested review from kra-mo and marcoambrosini and removed request for artonge July 31, 2025 10:43
@susnux
Copy link
Contributor Author

susnux commented Jul 31, 2025

@kra-mo this restores the icon as you said here: #53701 (comment)

app icons in the header navigation are excluded from the outlined icons
change, so this reverts this change.

Signed-off-by: Ferdinand Thiessen <[email protected]>
@susnux susnux force-pushed the fix/icons-header-meu branch from 3350052 to 5e979b5 Compare July 31, 2025 13:43
@susnux
Copy link
Contributor Author

susnux commented Jul 31, 2025

/compile

Signed-off-by: nextcloud-command <[email protected]>
@nextcloud-command nextcloud-command requested a review from a team as a code owner July 31, 2025 14:18
@susnux susnux merged commit b42f6bd into master Jul 31, 2025
133 of 136 checks passed
@susnux susnux deleted the fix/icons-header-meu branch July 31, 2025 16:18
@skjnldsv skjnldsv mentioned this pull request Aug 19, 2025
@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

Labels

3. to review Waiting for reviews bug design Design, UI, UX, etc.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants