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

[BUG]: No background in tooltips and menus #2713

Open
dcl opened this issue Jun 20, 2024 · 11 comments
Open

[BUG]: No background in tooltips and menus #2713

dcl opened this issue Jun 20, 2024 · 11 comments
Assignees
Labels
bug Something isn't working high priority

Comments

@dcl
Copy link

dcl commented Jun 20, 2024

Describe the bug

The background is missing in tooltips and and all other popovers, resulting in the content overlapping with whatever is underneath. The issue is appearing for me in both light and dark mode on the Windows client, but not in the browser.

Steps to Reproduce

  1. Install Windows client
  2. Start new project
  3. Open settings menu

Expected Behavior

The content in popovers is legible

Screenshots and Recordings

ss (2024-06-19 at 06 41 05)
ss (2024-06-19 at 06 41 14)
ss (2024-06-19 at 06 41 30)

Desktop OS

Windows 10

Browser

No response

Version

0.22.2

Additional Context

No response

@dcl dcl added the bug Something isn't working label Jun 20, 2024
@franknoirot
Copy link
Collaborator

This is fascinating. I suspect Windows 10 for some reason. I'll try my 11 machine and then see how I can boot up into 10.

@dcl would you happen to have any OS settings in effect, like setting everything to "high contrast mode", by any chance?

@franknoirot
Copy link
Collaborator

Oh you know what, I bet the background colors are failing because the platform OS for Windows 10, which would be used by Tauri, doesn't support OKLCH colors! I can try to replicate this on that route, turning off OKLCH color support and seeing if that does this. I think we need to provide fallback colors is all

@dcl
Copy link
Author

dcl commented Jun 21, 2024

This is fascinating. I suspect Windows 10 for some reason. I'll try my 11 machine and then see how I can boot up into 10.

@dcl would you happen to have any OS settings in effect, like setting everything to "high contrast mode", by any chance?

I don't have high contrast mode on, or afaict any other OS setting that should affect it. That said I suspect you're on to something with the color support, trying in dark mode introduces quite a lot more color issues than just the backgrounds:
ss (2024-06-20 at 06 11 07)

@jessfraz jessfraz added this to the v1 Modeling App Launch milestone Jun 21, 2024
@franknoirot
Copy link
Collaborator

That said I suspect you're on to something with the color support, trying in dark mode introduces quite a lot more color issues than just the backgrounds

Bingo, because in light mode the colors are falling back to black but that's not that different from our text color and whatnot, but in dark mode the OKLCH-defined text colors are falling back from off-white to black, looking way more jarring. We need some backup colors. I just need to figure out how far back up, like are we talking OKLCH isn't supported (relatively recent) or CSS variables (which have been around for a minute and we rely pretty heavily on). I'll try to make fallback behavior for both

@franknoirot
Copy link
Collaborator

Actually just one more question @dcl, could you provide me a version number of Edge that you're running? CSS Variables are supported all the way back to MS Edge v16 so I think I can get away with assuming a user has that, since WebRTC wasn't supported until v79 and it's a hard requirement for us. OKLCH wasn't supported until v111 so I think that's the culprit.

@dcl
Copy link
Author

dcl commented Jun 24, 2024

Actually just one more question @dcl, could you provide me a version number of Edge that you're running?

Great question! It seems to be v44? I have literally never run Edge before, it might still be old Edge?

@franknoirot
Copy link
Collaborator

I have literally never run Edge before

That would make sense then that you still have old Edge, and the thing about Tauri (how we build the desktop app) is it defers to the OS about what browser it should use, so it uses Edge on Microsoft of course, even if you as the user set Firefox or Chrome as your default browser.

If you don't mind, could you leave Edge in its old version for a bit? Once we get the PR I've made above in and release a new version, I'd love to see if you get solid backgrounds and better colors afterward.

Although I don't know if you'll be able to use the WebRTC stream we use to connect to the engine so after you verify that you might want to upgrade your Edge to a newer version sorry 😅

@dcl
Copy link
Author

dcl commented Jun 25, 2024

No worries, I'll hold off until we get this resolved.

Although I don't know if you'll be able to use the WebRTC stream we use to connect to the engine

Weirdly enough it seems to work fine. I wonder if Tauri has a fallback implementation?

@franknoirot
Copy link
Collaborator

Huh super weird I don't know! Neat that that part works though. We've just released v0.22.7 that includes a compatibility fallback color system to hsl that should work for you; do you mind updating to that version to see if it fixes your issue? You should see an updater popup if you open the app.

@dcl
Copy link
Author

dcl commented Jul 2, 2024

Alas, that doesn't seem to have fixed it. Same machine, 0.22.7:
ss (2024-07-01 at 05 07 56)

@franknoirot
Copy link
Collaborator

Dang it! Sorry for the false hope. I've just gotta download an old version of Chrome so I can see it breaking and then being fixed with my own eyes. We'll get there, apologies for the delay.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working high priority
Projects
None yet
Development

No branches or pull requests

3 participants