Open
Conversation
Fix skin section buttons disappearing when searching for plural "skins"
…tions Localise some more notification/updater strings
Addresses concerns in #36193 (comment) (excluding the last part, that is too involved and I can't imagine any workaround for it due to how strict the `Dropdown` structure is). Also adds truncation/padding to header label and search bar. Preview: https://github.com/user-attachments/assets/8885cb90-44dc-42ee-af21-cb33f7723e63
Use new song select (v2) for playlists
…rovided provider property The property used here is listed in SignalR documentation: https://learn.microsoft.com/en-us/aspnet/core/signalr/authn-and-authz?view=aspnetcore-10.0#bearer-token-authentication While in practice this probably has zero bearing on anything, theoretically the way proposed in this commit is more correct. As the documentation states, with some transports the token may need to be renewed if it expires, which providing it via a header as done previously would not achieve, while going through `API.AccessToken` every time will perform a token refresh if one is needed.
Clean-up from an old change (see #28892 (comment)).
More clean-up.
Clean up `HubClientConnector` configuration
Should've been true by default, minor mistake from a previous PR.
Used for "confine mouse mode" dropdown tooltip text.
Background blur is disabled by default.
- Related to ppy/osu-server-spectator#406 Adding this field to this model has several vague reasons that I can't fully formulate yet, but I can't really see myself going forward *without* this. - People were very excited about having referees displayed on the room participants' list, and so adding the referees as real `MultiplayerRoomUser`s helps this. Having the role could even be used client-side to show a special icon or other status on the participants list. (Which isn't done yet, could be as an aesthetic follow-up after the basics are in place.) - Server-side, having this field is convenient for things like permission checks or just plain logic, as with two hubs you just need to do different *stuff* on a `MultiplayerRoomUser`.
Fixes incorrect glow corner radius around user tag buttons Before: <img width="267" height="123" alt="image" src="https://github.com/user-attachments/assets/3e602e84-bb13-46f7-942c-85ddf3954946" /> After: <img width="227" height="102" alt="image" src="https://github.com/user-attachments/assets/4dca2e3d-80e2-4b6c-988e-d14f371cfbe8" />
…ps (#36663) Because people get confused by how this works. Shows on results screen where the post-play statistics updates go. https://github.com/user-attachments/assets/ef3a91d1-86dd-4029-8f0f-bdf0b727ca6c
Closes #33395 Copies the bookmarks from `referenceWorkingBeatmap` while creating a new difficulty from scratch. I adapted the tests in `TestSceneEditorBeatmapCreation` to include the bookmark checks. --------- Co-authored-by: Dean Herbert <pe@ppy.sh>
Resolves #36288. If the current selection is still available after leaving scoped mode, it's left as is. If it's not, the selection from before entering scoped mode is restored. https://github.com/user-attachments/assets/b1ac3de1-7c7f-4949-82a9-1dd0459f3f61 --------- Co-authored-by: Bartłomiej Dach <dach.bartlomiej@gmail.com>
…lected object is changed (#36681) Before: https://github.com/user-attachments/assets/d0a0373d-2d46-48a9-9ea5-bac82a612f32 After: https://github.com/user-attachments/assets/f5785b54-d7fc-4ce0-86b0-60c96ff22bc3 --- Closes #36677. Kinda shocking this went by unnoticed for this long.
In stable, there was support for mixed fonts (some coming from the user's skin, with fallback missing characters generated from TTF backend). We don't really have support for this so this is a simple in-between for the time being.
Matches closer to stable spec.
Fix legacy key counter not using skin's font
Of note, this test scene still doesn't use the new footer properly. Should these even be here? Maybe it's better to leave these real world flows to a `Navigation` style test so we're not dealing with weird test setups that kinda do something but don't... or maybe do it in three different ways (two footers present, three back buttons etc.) Dunno. But this kind of covers what I want (except it doesn't cover the new footer usage).
Add ability for `LoadingLayer` to block all keyboard input
- The mods button should not be visible because it is not hooked up properly to actually work with selecting user mods. As written it would show (and feign the ability to modify) required mods on the playlist item. You could probably adjust the logic to make user mods controllable via this button but this is simpler for now. - The random and options buttons could perhaps remain, but they are of very limited use on this screen anyway. The previous iteration disabled all footer buttons altogether, so I'm just following precedent here mostly.
Migrate playlist freestyle select screen to use `SongSelectV2`
Migrate multiplayer freestyle select screen to use `SongSelectV2`
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
20 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
See Commits and Changes for more details.
Created by
pull[bot]
Can you help keep this open source service alive? 💖 Please sponsor : )