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 ability to set guest name in lobby #3133

Merged

Conversation

marcoambrosini
Copy link
Member

Signed-off-by: Marco Ambrosini [email protected]

@marcoambrosini marcoambrosini added this to the 💚 Next Major (19) milestone Mar 23, 2020
@marcoambrosini marcoambrosini self-assigned this Mar 23, 2020
@marcoambrosini marcoambrosini force-pushed the bugfix/855/add-ability-to-set-guest-name-in-lobby branch from c73f1c4 to f6b2b1b Compare March 23, 2020 14:11
@marcoambrosini marcoambrosini marked this pull request as ready for review March 23, 2020 14:20
@marcoambrosini marcoambrosini changed the title Generalise guest rename logic into SetGuestUsername component for reuse Add ability to set guest name in lobby Mar 23, 2020
src/components/LobbyScreen.vue Outdated Show resolved Hide resolved
Copy link
Member

@nickvergessen nickvergessen left a comment

Choose a reason for hiding this comment

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

Feels a bit awkward that when you are typing your name and the lobby ends it is canceled.
But blocking the user from knowing the call started just because they opened the edit field is also weird.

@nickvergessen nickvergessen merged commit 83ec58d into master Mar 24, 2020
@nickvergessen nickvergessen deleted the bugfix/855/add-ability-to-set-guest-name-in-lobby branch March 24, 2020 07:35
@jancborchardt
Copy link
Member

jancborchardt commented Mar 27, 2020

This was btw in the spec for the lobby ;) #1926 (comment)

  • You can set your name already before you join the meeting.
    Talk lobby

@marcoambrosini
Copy link
Member Author

marcoambrosini commented Mar 27, 2020

Wow, that looks so different from what we have now... For the renaming I just copied the same component that we use in the sidebar for now, we did it together in January's contributor week @jancborchardt. Don't you think that it's better than the explicit input field??

Peek 2020-03-27 11-19

@jancborchardt
Copy link
Member

@ma12-co the idea was that if the name is the default "Guest", you probably want to change it and we can directly show the input field. It wouldn’t need to show as input field if you already set a name. My only suggestions for that:

  • We should call it "Your name" instead of "Display name" as that’s quite technical
  • The input field could replace the current name and the edit button in place, especially

Wow, that looks so different from what we have now...

Yeah – it’s rather bland at the moment. I’d still love for the mockup and spec to be followed a bit more:

  • It looks much much better. We don’t necessarily need to have the dark background, but this standard blue header and the white content just looks so basic.
  • Gives you feedback as to who you will be talking to – just like e.g. Hangouts does before joining
  • Shows your own camera in case it’s a video call – also just like Hangouts
  • Unclear what "waiting in the lobby" means. "Waiting for moderators to start the call" is clear and you know who to talk to if it does not start.
  • Some other points in the spec as well – give it a read at ⏳ Lobby #1926 (comment)

@marcoambrosini
Copy link
Member Author

Ok created an issue from your comment

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.

3 participants