Skip to content

First pass adjustments to new settings design#36486

Merged
bdach merged 23 commits intoppy:masterfrom
peppy:ui-tweaking
Jan 27, 2026
Merged

First pass adjustments to new settings design#36486
bdach merged 23 commits intoppy:masterfrom
peppy:ui-tweaking

Conversation

@peppy
Copy link
Copy Markdown
Member

@peppy peppy commented Jan 27, 2026

See individual commits for what has changed. I have more changes coming but these seem to stand alone quite amicably.

Release Tachyon This PR
osu! 2026-01-27 at 07 54 54 osu! 2026-01-27 at 07 54 06 osu! 2026-01-27 at 07 53 19
osu! 2026-01-27 at 07 57 48 osu! 2026-01-27 at 07 58 36 osu! 2026-01-27 at 08 06 44
osu Game Tests 2026-01-27 at 08 09 54 osu Game Tests 2026-01-27 at 08 09 01 osu Game Tests 2026-01-27 at 08 08 25

@peppy peppy added type/cosmetic Only affects the game visually. Doesn't affect things working or not working. area:settings labels Jan 27, 2026
@peppy peppy moved this from Inbox to Pending Review in osu! team task tracker Jan 27, 2026
@peppy peppy requested a review from bdach January 27, 2026 08:12
}
else
content.BorderColour = Enabled.Value ? colourProvider.Light4 : colourProvider.Dark1;
// TODO: Support BackgroundColour?
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

is this TODO still supposed to be here?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Yeah. this was added by @frenzibyte but is not used yet, so I'm not sure how/when to support it. In addition FormButton doesn't really work in settings yet.

@diquoks

This comment was marked as off-topic.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:settings size/XL type/cosmetic Only affects the game visually. Doesn't affect things working or not working.

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

3 participants