Skip to content

Conversation

@szaimen
Copy link
Contributor

@szaimen szaimen commented Oct 14, 2022

Close #34656

Before After
image image

Signed-off-by: szaimen szaimen@e.mail.de

@szaimen szaimen added bug design Design, UI, UX, etc. 3. to review Waiting for reviews labels Oct 14, 2022
@szaimen szaimen added this to the Nextcloud 26 milestone Oct 14, 2022
@szaimen

This comment was marked as resolved.

@szaimen

This comment was marked as resolved.

@CarlSchwan

This comment was marked as resolved.

@szaimen szaimen added 2. developing Work in progress and removed 3. to review Waiting for reviews labels Oct 18, 2022
@szaimen

This comment was marked as resolved.

@jancborchardt

This comment was marked as resolved.

@szaimen

This comment was marked as resolved.

@skjnldsv skjnldsv self-assigned this Nov 8, 2022
@skjnldsv skjnldsv force-pushed the enh/noid/logo-inverted branch from 4489f28 to 975174a Compare November 8, 2022 16:04
@skjnldsv
Copy link
Member

skjnldsv commented Nov 8, 2022

Done, please check @szaimen

@skjnldsv skjnldsv force-pushed the enh/noid/logo-inverted branch from 975174a to 26ea38d Compare November 8, 2022 16:05
@skjnldsv skjnldsv added 3. to review Waiting for reviews and removed 2. developing Work in progress labels Nov 8, 2022
@skjnldsv skjnldsv requested a review from CarlSchwan November 8, 2022 16:36
Copy link
Contributor Author

@szaimen szaimen left a comment

Choose a reason for hiding this comment

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

This doesnt seem to work correctly yet.
Just tested and it looks like this now with dark mode enabled:
image

@szaimen
Copy link
Contributor Author

szaimen commented Nov 15, 2022

@skjnldsv sorry for bothering you but could you maybe take another look at this? I think we need to get the value from the our default themes by checking against its dark value...

Signed-off-by: John Molakvoæ <skjnldsv@protonmail.com>
@skjnldsv skjnldsv force-pushed the enh/noid/logo-inverted branch from 22fe22e to e6b7fa6 Compare November 16, 2022 08:46
@skjnldsv
Copy link
Member

@skjnldsv sorry for bothering you but could you maybe take another look at this? I think we need to get the value from the our default themes by checking against its dark value...

My bad, wrong fallback value! 👍
Fixed!

Copy link
Contributor Author

@szaimen szaimen left a comment

Choose a reason for hiding this comment

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

Seems to work correclty now in my testing so ready to go from my side! :)
Unfortunately I cannot approve as it is my PR but you could approve for me :P

Copy link
Member

@PVince81 PVince81 left a comment

Choose a reason for hiding this comment

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

👍

@szaimen szaimen merged commit 3d77e38 into master Nov 16, 2022
@szaimen szaimen deleted the enh/noid/logo-inverted branch November 16, 2022 14:54
@szaimen
Copy link
Contributor Author

szaimen commented Nov 16, 2022

/backport to stable25

@backportbot-nextcloud
Copy link

The backport to stable25 failed. Please do this backport manually.

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.

[Bug]: Background pictures which invert the heading icons do not invert the logo

6 participants