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

Dim static elements in screensaver #3368

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

abjorck
Copy link

@abjorck abjorck commented Feb 11, 2024

I'd like to "reopen" pull #2566 to dim the white static elements on the screensaver

Changes

Set alpha on logo, clock, and content title.

I also did a miniscule change to the fade since I think the snap-out was sometimes off and gave a janky feeling when the old image disappeared, not sure why, maybe a frame timing thing. Seems hard to record and report but I can try making it a seperate issue if you dont want it here

Issues

@abjorck abjorck force-pushed the Disable_static_elements_on_screensaver branch from 8d4157e to fa37f5a Compare February 15, 2024 21:33
@jellyfin-bot jellyfin-bot added the merge conflict Conflicts prevent merging label Mar 11, 2024
Copy link
Member

@nielsvanvelzen nielsvanvelzen left a comment

Choose a reason for hiding this comment

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

Somewhat dimming the text is fine but the Jellyfin logo should not be made semi-transparent.

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

Successfully merging this pull request may close these issues.

None yet

3 participants