Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Notifications menu and contact-search menu broken with certain predefined backgrounds #42962

Closed
nickvergessen opened this issue Jan 19, 2024 · 12 comments · Fixed by #42963
Closed

Comments

@nickvergessen
Copy link
Member

How to use GitHub

  • Please use the 👍 reaction to show that you are interested into the same feature.
  • Please don't comment if you have no relevant information to add. It's just extra noise for everyone subscribed to this issue.
  • Subscribe to receive notifications on status change and new comments.

Steps

  1. Go to go to index.php/settings/user/theming
  2. Select one of the following default offered backgrounds: All that make the font color go black
  • Pink bubbles
  • Yellow bricks
  • Lion rock hill
  • "the ducks"
  • soft floral
  • morning fog
  1. Open the notification menu => Inverted color ❌
  2. Open the contact search => menu opens in place of the button instead of a popover menu 💥
    Bildschirmfoto vom 2024-01-19 14-43-44

How it looks with any other background:

Bildschirmfoto vom 2024-01-19 14-43-35

@ShGKme
Copy link
Contributor

ShGKme commented Jan 19, 2024

While filter inverts the icon, it also breaks the whole header menu.

Without filter With filter
image image

@szaimen
Copy link
Contributor

szaimen commented Jan 19, 2024

Yeah, I wonder how a simple filter can do this??? Browser bug maybe?

@ShGKme
Copy link
Contributor

ShGKme commented Jan 19, 2024

Yeah, I wonder how a simple filter can do this??? Browser bug maybe?

Anyway, we should not invert a whole menu. Only the icon.

@szaimen
Copy link
Contributor

szaimen commented Jan 19, 2024

Anyway, we should not invert a whole menu. Only the icon.

I agree.

@szaimen szaimen self-assigned this Jan 19, 2024
@szaimen szaimen added 2. developing Work in progress and removed 1. to develop Accepted and waiting to be taken care of labels Jan 19, 2024
@szaimen
Copy link
Contributor

szaimen commented Jan 19, 2024

Fix is in #42962

I am not really satisfied with the solution but it is what it is...

@blizzz
Copy link
Member

blizzz commented Jan 19, 2024

Fix is in #42962

Or maybe #42963

@szaimen
Copy link
Contributor

szaimen commented Jan 19, 2024

Fix is in #42962

Or maybe #42963

Yes, sorry

@ChristophWurst
Copy link
Member

It's not just a visual bug. I see a network request to load the content of the menu even before it opens. That should only happen on click.

@ChristophWurst
Copy link
Member

ChristophWurst commented Jan 22, 2024

Test master. The menu is still as broken as it was before for me:

Bildschirmfoto vom 2024-01-22 12-07-07

The POST /contactsmenu/contacts also happens after page load, before I event click on the contacts menu toggle.

@szaimen
Copy link
Contributor

szaimen commented Jan 22, 2024

@ChristophWurst apparently someone forgot to recompile css and our CI didnt catch that... See #43017

@ChristophWurst
Copy link
Member

The POST /contactsmenu/contacts also happens after page load, before I event click on the contacts menu toggle.

Is still there, even after recompilation.

@szaimen
Copy link
Contributor

szaimen commented Jan 22, 2024

The POST /contactsmenu/contacts also happens after page load, before I event click on the contacts menu toggle.

Is still there, even after recompilation.

See #43018

@szaimen szaimen closed this as completed Jan 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

6 participants