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

Improve Menu Contrast and Brightness in Dark Mode #5446

Closed
Bonapara opened this issue May 16, 2024 · 9 comments
Closed

Improve Menu Contrast and Brightness in Dark Mode #5446

Bonapara opened this issue May 16, 2024 · 9 comments
Assignees
Labels
good first issue Good for newcomers scope: front Issues that are affecting the frontend side only size: minutes

Comments

@Bonapara
Copy link
Member

Bonapara commented May 16, 2024

Current Behavior

In dark mode, the menu borders are currently darker than the content, and the menu background appears too light.

Contrast is set to 50% in both light and dark mode

backdrop-filter: blur(12px) saturate(200%) contrast(50%) brightness(130%);

image

Desired Behavior

Contrast should be set to 100% in dark mode:

backdrop-filter: blur(12px) saturate(200%) contrast(100%) brightness(130%);
image
@Bonapara Bonapara added good first issue Good for newcomers scope: front Issues that are affecting the frontend side only size: minutes labels May 16, 2024
Copy link

greptile-apps bot commented May 16, 2024

Disclaimer: This comment was automatically generated with AI. It can give some useful hints as to where to look for, but sometimes it can also give very inaccurate answers so takes this with a pinch of salt :).
To improve the menu contrast and brightness in dark mode, update the CSS properties in the relevant theme files:

  1. Modify OverlayBackground.ts to include the desired backdrop-filter properties for dark mode:
// OverlayBackground.ts
const darkModeStyles = {
  backdropFilter: 'blur(12px) saturate(200%) contrast(100%) brightness(130%)',
  // other styles
};
  1. Ensure BackgroundDark.ts and BackgroundLight.ts are consistent with the new design requirements.

References

@Bonapara
Copy link
Member Author

@charlesBochet, wouldn't this break the design system? How would you implement it?

@its-id
Copy link
Contributor

its-id commented May 16, 2024

I think I might have done similar fixes in the past. Awaiting here.

Can I take a look?

@FelixMalfait
Copy link
Member

Thanks @its-id! Assigning you

@Bonapara
Copy link
Member Author

Hi @its-id are you still on it?

@its-id
Copy link
Contributor

its-id commented May 28, 2024

Hi,
really sorry for the delay. was busy for the past weeks due to personal reason. will complete by today.

@Bonapara
Copy link
Member Author

Thanks @its-id 🙏

@its-id
Copy link
Contributor

its-id commented May 30, 2024

I think this issue has already been fixed 3 days ago.
335357013-661c8676-8a74-4b4f-91f8-d3ab3f1a00ea
Screenshot 2024-05-31 at 1 13 11 AM
image

can you confirm? @Bonapara

once again, am really sorry for the delay. will not happen again.

@Bonapara
Copy link
Member Author

Don't worry, and thanks for contributing! Yes, it seems to have been fixed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Good for newcomers scope: front Issues that are affecting the frontend side only size: minutes
Projects
Status: ✅ Done
Development

No branches or pull requests

3 participants