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.

Sign up for GitHub

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

Jump to bottom

[FR]: communicate users that a password is optional for new profiles #299

Closed
1 task done
Chiitoo opened this issue Dec 24, 2024 · 3 comments · Fixed by #347
Closed
1 task done

[FR]: communicate users that a password is optional for new profiles #299

Chiitoo opened this issue Dec 24, 2024 · 3 comments · Fixed by #347
Labels
enhancement New feature for the user, not a new feature for build script
Milestone
v1.18.1

Comments

@Chiitoo
Copy link

Chiitoo commented Dec 24, 2024

Contact Details (e.g. Tox ID or email)

No response

Feature Description

While creating a new profile, there is nothing indicating that it is not necessary to enter a password, and that it is not possible to recover it, nor that it is only stored locally.

Perhaps there could at least be a tooltip (or a question-mark-button type of a thing) briefly explaining these things.

Contribution

  • I would like to contribute to this feature
@Chiitoo Chiitoo added the enhancement New feature for the user, not a new feature for build script label Dec 24, 2024
@iphydf iphydf added this to the v1.18.1 milestone Dec 24, 2024
@iphydf
Copy link
Member

iphydf commented Jan 2, 2025 •
edited
Loading

image

I tried this to be more clear. I think it's ugly, so I'll go with a tooltip directly on the password field, instead, so when people hover, they'll see the message.

@iphydf
Copy link
Member

iphydf commented Jan 2, 2025

Tooltip:
image

WhatsThis:
image

iphydf added a commit to iphydf/qTox that referenced this issue Jan 2, 2025
@iphydf
feat(Login): Add a tooltip and WhatsThis for the password field.
0938017 
This explains that passwords are optional, local, can't be recovered,
and aren't sent to servers.

Fixes TokTok#299.
@iphydf iphydf mentioned this issue Jan 2, 2025
Merged
iphydf added a commit to iphydf/qTox that referenced this issue Jan 2, 2025
@iphydf
feat(Login): Add a tooltip and WhatsThis for the password field.
598b108 
This explains that passwords are optional, local, can't be recovered,
and aren't sent to servers.

Fixes TokTok#299.
iphydf added a commit to iphydf/qTox that referenced this issue Jan 2, 2025
@iphydf
feat(Login): Add a tooltip and WhatsThis for the password field.
de5198c 
This explains that passwords are optional, local, can't be recovered,
and aren't sent to servers.

Fixes TokTok#299.
iphydf added a commit to iphydf/qTox that referenced this issue Jan 2, 2025
@iphydf
feat(Login): Add a tooltip and WhatsThis for the password field.
7afb535 
This explains that passwords are optional, local, can't be recovered,
and aren't sent to servers.

Fixes TokTok#299.
iphydf added a commit to iphydf/qTox that referenced this issue Jan 2, 2025
@iphydf
feat(Login): Add a tooltip and WhatsThis for the password field.
13451c0 
This explains that passwords are optional, local, can't be recovered,
and aren't sent to servers.

Fixes TokTok#299.
iphydf added a commit to iphydf/qTox that referenced this issue Jan 2, 2025
@iphydf
feat(Login): Add a tooltip and WhatsThis for the password field.
4ff7438 
This explains that passwords are optional, local, can't be recovered,
and aren't sent to servers.

Fixes TokTok#299.
iphydf added a commit to iphydf/qTox that referenced this issue Jan 2, 2025
@iphydf
feat(Login): Add a tooltip and WhatsThis for the password field.
bfd5938 
This explains that passwords are optional, local, can't be recovered,
and aren't sent to servers.

Fixes TokTok#299.
@Chiitoo
Copy link
Author

Chiitoo commented Jan 2, 2025

Yeah, that looks good to me.

Thank you!

iphydf added a commit to iphydf/qTox that referenced this issue Jan 2, 2025
@iphydf
feat(Login): Add a tooltip and WhatsThis for the password field.
3fab032 
This explains that passwords are optional, local, can't be recovered,
and aren't sent to servers.

Fixes TokTok#299.
iphydf added a commit to iphydf/qTox that referenced this issue Jan 3, 2025
@iphydf
feat(Login): Add a tooltip and WhatsThis for the password field.
4e92ff9 
This explains that passwords are optional, local, can't be recovered,
and aren't sent to servers.

Fixes TokTok#299.
@iphydf iphydf closed this as completed in 04efcf8 Jan 3, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement New feature for the user, not a new feature for build script
Milestone
v1.18.1
Development

Successfully merging a pull request may close this issue.

feat(Login): Add a tooltip and WhatsThis for the password field. iphydf/qTox
2 participants
@Chiitoo @iphydf