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

Add ability to render frosted sliders #30347

Draft
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

minetoblend
Copy link
Contributor

@minetoblend minetoblend commented Oct 19, 2024

RFC. I took peppy's tweet as a challenge to see if I can get this effect working in lazer. Since the "someone chooses to implement" part is done, I guess it is now time to figure out the "maybe" part.

Depends on ppy/osu-framework#6393

2024-10-19.02-06-11.mp4

This is primarily meant a proof of concept, so I only implemented this for legacy skins for now.

There is a lot of uncertainty about how this would actually work as part of the game, ux and stuff. Would it be a configurable property in skins? Perhaps a custom skin type?

Right now I'm doing the blurring at 50% resolution (and drawing the content on full resolution), though I've also tested it at 33% and 25% resolution and didn't notice any huge impact visually with either. I tried to dial in some values for the effect that look pretty good for most maps, but some more adjustments are definitely still needed.

Some screenshots

image

image

image

@pull-request-size pull-request-size bot added size/S and removed size/M labels Oct 21, 2024
@peppy peppy changed the title Add ability to render translucent sliders Add ability to render frosted sliders Oct 22, 2024
@peppy
Copy link
Member

peppy commented Oct 23, 2024

Given that we've seen the blur shader cause performance issues on some devices in the past, doing this kind of effect on a gameplay element leaves me a touch cautious.

It's a very cool effect, but we'd want to make sure that this is done with performance in mind.

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

Successfully merging this pull request may close these issues.

2 participants