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 button in playlists/multiplayer to access the osu-web page #31006

Closed
wants to merge 3 commits into from

Conversation

Layendan
Copy link
Contributor

@Layendan Layendan commented Dec 8, 2024

2024-12-08_00-13-05.mp4

Add a button to multiplayer lobbies that allows users to open the room in the browser.

@Layendan Layendan marked this pull request as draft December 8, 2024 08:40
@Layendan Layendan marked this pull request as ready for review December 8, 2024 09:23
@Layendan
Copy link
Contributor Author

Layendan commented Dec 8, 2024

I thought this would only work on multiplayer and did not realize that it would also show in playlists. Tested on both and made sure that they all work. Should be good to review.

@smoogipoo
Copy link
Contributor

smoogipoo commented Dec 9, 2024

Design wise, is this what we really want to do @peppy (@ppy/team-design)? It feels shocking to me to add this random symbol everywhere in the game - is that just me?

I ask because this probably would guide other fixes. For example, the room name no longer truncates correctly:

master pr
Screenshot 2024-12-09 at 10 13 30 Screenshot 2024-12-09 at 10 12 45

Fixing the above is non-trivial due to the sizing situation involved.

@cl8n
Copy link
Member

cl8n commented Dec 9, 2024

I agree w/ you, on top of not looking great to have symbols anchored to the end of text like this, it feels like it's suggesting you can see something more on the web page, but that kind of begs the question of why it wouldn't be shown in the game in the first place..?

for a component like drawableroom, I think the "view in browser" and "copy link" buttons fit better as context menu options on the entire panel, no icon needed

@peppy
Copy link
Member

peppy commented Dec 9, 2024

The explicit icon should only be used in places that people are expected to regularly be copying links. In multiplayer, it's arguable that it should be there because in stable, for instance, the website shows history which isn't visible in the game. It's also useful for sharing the link in a tournament context.

It definitely should not show in every multiplayer game in the list.

Also I agree with context menu support. If we keep the icon in the multiplayer room header, I'd still see context menu support also added at the listing / room in addition, to give users one more way to do the same thing.

@Layendan
Copy link
Contributor Author

Layendan commented Dec 9, 2024

Yeah tbh I wasn't too happy myself at how the design ended up looking like. I think at this point it would be better to add the "view in browser" and "copy link" options to the context menu and remove the button entirely, but I would like to argue that it should be available everywhere (both listings and rooms).

@peppy
Copy link
Member

peppy commented Dec 10, 2024

but I would like to argue that it should be available everywhere (both listings and rooms).

as i said, in context menu is fine. but the icon should not display in a listing.

@smoogipoo
Copy link
Contributor

Let's close this then because it doesn't sound like anything is salvageable.

@smoogipoo smoogipoo closed this Dec 10, 2024
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