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

UI layer issue for collection assignment #21602

Closed
Inconsist opened this issue Dec 11, 2022 · 3 comments · Fixed by #29394
Closed

UI layer issue for collection assignment #21602

Inconsist opened this issue Dec 11, 2022 · 3 comments · Fixed by #29394
Assignees
Labels
area:song-select priority:1 Very important. Feels bad without fix. Affects the majority of users. type:cosmetic

Comments

@Inconsist
Copy link

Type

Cosmetic

Bug description

For long collection names the menu flips over and is then coverd by the UI that's already there. Would be nice to have it be atop for this so you could see what you're doing more easily. Also my specific collections are nolonger sortet in the same way in the dropdown as they are where you can rearrange them in the "Manage" panel.

Screenshots or videos

for the layer issue
image

the way I sort them
image
the way I see them
image

Version

2022.1208.0-lazer

Logs

input.log
network.log
performance.log
runtime.log
updater.log
database.log

@Inconsist
Copy link
Author

actually I figured out how to order collections, theyre still sortet alphabetically, you just have to press enter after a namchange. the hamburger drag and drop reorder thing has no effect, thats misleading design.

@sponnify
Copy link

sponnify commented Oct 14, 2023

hi, i’d like to add some context to this issue. as a mapper who heavily uses collections, these issues make it difficult to use lazer for editing beatmaps for my workflow.

i’ve noticed that new collections created in lazer’s manage collections window are not correctly organised alongside collections imported from the stable version. they appear at the bottom of the collections listing, regardless of their names or any symbols used. this isn’t due to exceeding stable’s character limit for collection names, as i’ve demonstrated in the following videos.

https://github.com/ppy/osu/assets/69046095/b30d623c-b8e2-4bea-9b9e-091f41fcf33f
https://github.com/ppy/osu/assets/69046095/077e1d89-896b-4324-bde4-74dd10a3922d

however, the manage collections window does display the collections in the correct order when opened. this discrepancy between the ordering in different parts of the ui is confusing and hinders efficient use of the collections feature. i agree that this is misleading and can be quite frustrating for users who rely on collections for their workflow.

@peppy
Copy link
Member

peppy commented Oct 16, 2023

i’ve noticed that new collections created in lazer’s manage collections window are not correctly organised alongside collections imported from the stable version

I've fixed this in #25139

I'm not going to attempt to look into the layering issue as I believe this should be considered / fixed by @Joehuu's reimplementation of song select.

@Joehuu Joehuu self-assigned this Dec 15, 2023
@peppy peppy added the priority:1 Very important. Feels bad without fix. Affects the majority of users. label Aug 12, 2024
@peppy peppy assigned peppy and unassigned Joehuu Aug 12, 2024
peppy added a commit to peppy/osu that referenced this issue Aug 12, 2024
peppy added a commit to peppy/osu that referenced this issue Aug 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area:song-select priority:1 Very important. Feels bad without fix. Affects the majority of users. type:cosmetic
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants