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

Shorter Teasers please #142

Closed
jauggy opened this issue Aug 2, 2024 · 11 comments
Closed

Shorter Teasers please #142

jauggy opened this issue Aug 2, 2024 · 11 comments

Comments

@jauggy
Copy link
Member

jauggy commented Aug 2, 2024

Sample teaser: Boss name | Custom Battle 8 vs 8 should become simply 8 vs 8. It's taking up too much space on lobby browser and other info is getting cut off.

Discord discussion
https://discord.com/channels/549281623154229250/549281623577722899/1269025159558332599

@DeviousNull
Copy link
Collaborator

#145 should help with this a little, but I could be more aggressive with shortening it too.

What do you think of the following?

  1. If the lobby is empty/private then set the Teaser to empty
  2. Otherwise, if there is a Boss then set the teaser to Boss: NAME (or even 👑NAME, if the unicode-crown looks good in-game)
  3. Otherwise, set the teaser based on the current preset: Team 8v8 / Coop vs AI / Tourney / Custom 8v8

That way there would be at most one component added to the title by the Teaser

@DeviousNull
Copy link
Collaborator

After some brief testing, the 👑NAME idea seems non-trivial. It looks like Chobby doesn't appreciate Unicode characters in lobby names:
2024-08-04 17_51_45

So, I'm just going to update my existing PR to implement the logic mentioned above; that may reduce the number of characters enough.

@jauggy
Copy link
Member Author

jauggy commented Aug 5, 2024

Based on the discord discussion: https://discord.com/channels/549281623154229250/549281623577722899/1269025159558332599

Many people just want 8v8 and nothing else. We don't need Team or Custom Battle. For the boss name it can take up a lot of space, but maybe just put name or just put Boss.

@FIr3baL any reason why we can't show symbols in the lobby title? I know that σ symbol for sigma works in other areas for Chobby.

@FIr3baL
Copy link
Contributor

FIr3baL commented Aug 5, 2024

Based on the discord discussion: https://discord.com/channels/549281623154229250/549281623577722899/1269025159558332599

Many people just want 8v8 and nothing else. We don't need Team or Custom Battle. For the boss name it can take up a lot of space, but maybe that is something Chobby can handle for people with small screens i.e. me.

@FIr3baL any reason why we can't show symbols in the lobby title? I know that σ symbol for sigma works in other areas for Chobby.

Since chobby is written in LUA, it should be complete compatible with UTF-8.
I guess it's a transport problem.
You could try changing c.battle.update_lobby_title and s.battle.update_lobby_title to use base64 encoded strings to bypass any problems, that can occur with UTF-8. Unfortunatly this would need to be changed in teiserver+barmanager+chobby.

@jauggy
Copy link
Member Author

jauggy commented Aug 5, 2024

Yeah probably not worth it if we have to do that. But thanks.

@MitjaGrabbe
Copy link

Hello, as Jauggy mentioned me as a frequent lobby hoster would prefer 8v8 without a boss name or custom game or teamgame prefix. People are mostly interested in the min rating or min chev level which is currently for most players not visible.

I host lobbies often and I have to say that this is one of the changes I do really dislike. to be precise I just stopped hosting lobbies as it is way to hard to create lobby names that read pleasant for the eye, hence people join. of course glitter lobbies etc dont face that issue.

Even if it isnt a custom game and so a proper ranked and the boss name then is shown in the back, it is still spam. Talked to many players. We seems to agree that this change isnt perceived as good. I rather would welcome a popup if I hover over a lobby with my mouse, or completly new collums in the lobby list seperated as maps currently are shown.

@jauggy
Copy link
Member Author

jauggy commented Aug 5, 2024

Near and Kyouma and also agreed that boss is not necessary in the title https://discord.com/channels/549281623154229250/549281623577722899/1269025823395024988

Choice is also not a fan of the current naming scheme:
https://discord.com/channels/549281623154229250/549281623577722899/1269055882797518939

@DeviousNull
Copy link
Collaborator

Beherith already merged the last PR, but I can put up another one later today when I have some time to work on it. Think this would work?

If bots are present: | PRESET_NAME vs BOT_TYPE
If no bots are present:

  • Team/Custom: only team-size * number-of-teams (i.e. | 8v8)
  • FFA/Duel/Tourney/Coop: | PRESET_NAME

@jauggy
Copy link
Member Author

jauggy commented Aug 6, 2024

@DeviousNull sounds good

@DeviousNull
Copy link
Collaborator

All of my pending PRs to shorten teasers have been merged now, and most lobbies have updated to use the new names (the few remaining should get the update over the next few days or so).

If we want to go a few characters shorter, we could consider shortening the Min chev: / Min rating: prompts as well:

  • Max only: Rank: 0-2 / Skill: 0-10
  • Min+Max: Skill 10-20
  • Min only: Rank: 2+ / Skill 20+

... but I'm not sure that's really necessary at this point.

@jauggy
Copy link
Member Author

jauggy commented Aug 9, 2024

OK I see some good improvements in production now. Closing.

For further improvements I've got a ticket in chobby for allowing the length of the lobby browser to be wider: beyond-all-reason/BYAR-Chobby#743

Currently you can go to Settings> Split Panel > Always One to make it wider but it also affects when you are in lobby.

@jauggy jauggy closed this as completed Aug 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants