-
Notifications
You must be signed in to change notification settings - Fork 452
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
Use a more descriptive displayId #6437
Conversation
IIRC we avoid using me/you as much as possible in the UI. |
Also it's the reverse of the Avatar menu action |
Ah wait, it's only the label of the settings option? |
Yep, @nextcloud/designers feel free to adjust :)
Currently yes, just used when the user configures visibility |
e1452b5
to
c232ba5
Compare
Signed-off-by: Christopher Ng <chrng8@gmail.com>
c232ba5
to
addb531
Compare
Btw being able to set it to "Everyone" although it's maximum is "logged in users" might lead to some confusion in the future. But fine for now I guess |
Use a more descriptive displayId for the profile action, "Reach me via Talk" instead of just "Talk", so that it's clearer when users are configuring their profile settings, as suggested by @nimishavijay
i.e.
