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

Windows Acrylic Background #577

Open
wants to merge 11 commits into
base: main
Choose a base branch
from

Conversation

abhiroopc84
Copy link
Contributor

Description

This PR includes background changes that make the acrylic background possible on Windows. It might work on MacOS as well if there are no unintended side-effects.
Screenshot (476)
Screenshot (477)

What is the purpose of this pull request?

  • New feature
  • Documentation update
  • Bug fix
  • Refactor
  • Release
  • Other

@drfarrell drfarrell self-requested a review October 17, 2024 21:41
Copy link
Collaborator

@drfarrell drfarrell left a comment

Choose a reason for hiding this comment

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

Feedback on the acrylic:
WOAH this looks sick! It works on Mac too. It's very subtle which is so nice. Super cool!

Feedback on the layers panel

  • It seems like the layers panel background is too transperant. Maybe things were changed when you added in the acrylic effect, but it's hard to read the text on the sidepanel now. See below.
  • It seems like the layer's have an abupt ending depending on the length of the string or the string continues too far. See the screenshot below:
image - The Icons look really good on light and dark. One thing for specifically selected classes, is that the icons also need to be white / active or whatever the class is along with the text. See if you can add that class in for that icon. Screenshot 2024-10-17 at 5 46 33 PM

@abhiroopc84
Copy link
Contributor Author

I'll have a look at the layers panel background and see why it's abruptly ending as well.

As for the icons, I was wondering how I should style the header icons because the H and the layer number are coloured differently so, when the icon is active, do I colour both the H and the number active with the same colour?

@drfarrell
Copy link
Collaborator

I'll have a look at the layers panel background and see why it's abruptly ending as well.

As for the icons, I was wondering how I should style the header icons because the H and the layer number are coloured differently so, when the icon is active, do I colour both the H and the number active with the same colour?

I think for the case when they are selected they can both be the same color (white/active) just for maximum contrast against the color background. When they aren't selected, I like the half-tone coloring, but you tell me if that's too tough to engineer.

@abhiroopc84
Copy link
Contributor Author

@drfarrell I've fixed both, the issue where in the layers panel background was too transparent and also that of layers having an abrupt ending depending on the length of the string.

@Kitenite
Copy link
Contributor

Kitenite commented Oct 18, 2024

Performance didn't seem to degrade on Mac though I can't tell on my version. Otherwise looks good! Deferring to @drfarrell for the final review.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants