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 mod display and unranked indicator to new song select footer #28141

Merged
merged 13 commits into from
May 11, 2024

Conversation

frenzibyte
Copy link
Member

Preview:

CleanShot.2024-05-10.at.07.21.40.mp4
CleanShot.2024-05-10.at.07.34.46.mp4

Notes:

  • I've had to disable display of extended information for mods such as DT, as displaying only one of them eats up the whole display:

    CleanShot 2024-05-10 at 02 51 11

    I'm not sure what's the best way to display those, or if this indicates a flaw in the current design of the mod display (cc @peppy).

  • There is supposed to be a drop-shadow behind the background of the "mod display" part of the bar above the button, but when adding it, the bottom left edge of the display becomes overfilled with shadow, making it look ugly:
    CleanShot 2024-05-10 at 06 59 51
    It may not be as noticable (maybe because of the background being dark though). If there is nothing looking off with the image above then I can add the drop shadow (diff, for real testing).

  • There are different other ways for displaying the unranked indicator proposed in discord, such as adding a red outline to the mod display, or adding a warning icon next to the mod display with an explanatory tooltip. I'm not really a fan of any of those proposals as it sort-of defeats the purpose of the indicator, which is to make the user well informed about their mod selection without having them hover over it.

@cl8n
Copy link
Member

cl8n commented May 10, 2024

the right edge of the unranked indicator doesn't seem to align (in the sheared sense...) with the button below it

@peppy
Copy link
Member

peppy commented May 10, 2024

I've had to disable display of extended information for mods such as DT

Is it hard to have it display until it doesn't fit? And potentially in the tooltip?

I'm wondering if we need a compact version which shows that there are non-default settings while not being the full settings text (just a slightly extended colour bar).

@EVAST9919
Copy link
Contributor

There is supposed to be a drop-shadow behind the background of the "mod display" part of the bar above the button, but when adding it, the bottom left edge of the display becomes overfilled with shadow, making it look ugly

Figma uses gaussian blur for shadows while o!f uses linear fade (hacky repurposed masking shader). I have a way to fake gaussian blur for simple shapes without any overhead, but there's no way to change that without big refactoring unfortunately.

@frenzibyte
Copy link
Member Author

frenzibyte commented May 10, 2024

I've had to disable display of extended information for mods such as DT

Is it hard to have it display until it doesn't fit? And potentially in the tooltip?

It's possible (I tried and it worked), but with only one customised mod, adding any other mod will fall back to text mode, so I scrapped the idea of showing them in the first place altogether. The tooltip I added for "X mods" text already shows the mod in its extended form.

@Joehuu
Copy link
Member

Joehuu commented May 11, 2024

Might need #19750 to distinguish modified rate adjust mods without having the extended information state. Still want to see the rate multiplier though.

I feel like the tooltip for 5+ mods seems to be lacking information in it's current state. Individual mod tooltips show the mod name and settings modified but doesn't in this grouped 5+ mod tooltip. Was recommending to reuse or copy the mod preset tooltip that has this information w/ consistent mod icons.

Also TestSceneFooterButtonModsV2 crashes with that tooltip:

[runtime] 2024-05-11 04:07:12 [error]: An unhandled error has occurred.
[runtime] 2024-05-11 04:07:12 [error]: osu.Framework.Allocation.DependencyNotRegisteredException: The type FooterButtonModsV2+ModCountText+ModTooltip has a dependency on OverlayColourProvider, but the dependency is not registered.

@frenzibyte
Copy link
Member Author

I feel like the tooltip for 5+ mods seems to be lacking information in it's current state. Individual mod tooltips show the mod name and settings modified but doesn't in this grouped 5+ mod tooltip. Was recommending to reuse or copy the mod preset tooltip that has this information w/ consistent mod icons.

I went with a different design myself because the purpose of this tooltip is to show what the display couldn't due to its insufficient width. Adding extra information like the name next to the mod feels too much information to me, and even if we add it, it feels weird that we're only showing this extra information only when there are too many mods selected.

@peppy peppy self-requested a review May 11, 2024 12:17
Copy link
Member

@peppy peppy left a comment

Choose a reason for hiding this comment

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

one step forward

@peppy peppy merged commit 6cfe135 into ppy:master May 11, 2024
13 of 17 checks passed
@frenzibyte frenzibyte deleted the footer-v2-mods-display branch May 11, 2024 13:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants