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

Textbox inputs become disabled after entering an incorrect password when clicking the enable encryption button in Encryption settings #10887

Open
mrjo118 opened this issue Aug 16, 2024 · 4 comments · May be fixed by #11241
Labels
bug It's a bug

Comments

@mrjo118
Copy link

mrjo118 commented Aug 16, 2024

Operating system

Windows

Joplin version

3.0.14

Desktop version info

Windows 10

Current behaviour

-Start with a brand new profile in Joplin desktop
-Go to tools, manage master password and set any password
-Go to tools, options, encryption and click enable encryption
-Enter an incorrect password and click ok. Then click ok the error message that pops up
-Attempt to click enable encryption and enter the password again, but you cannot because the textbox input is disabled. Go out of the settings screen and find text boxes everywhere are disabled
-Switch to another application on windows, and then back to Joplin, and find the textbox inputs magically enabled again

Expected behaviour

Textbox inputs should not be disabled after entering an incorrect password.

On a side note, if you enable encryption using the enable encryption button before a master password has been set, it creates a master password on the fly when you click ok. This seems wrong. I think it should in that scenario throw an error saying you must first set a master password, because it is kind of unsafe to enter a new password in a masked text input without a verification input as well, or the option to show the password on screen. If the user makes a mistake in entering the password, then they might not know until they have pushed all their notes to an online sync source, and then start trying to sync to another device and find the password does not work

Logs

No response

@mrjo118 mrjo118 added the bug It's a bug label Aug 16, 2024
Copy link
Contributor

Hey there, it looks like there has been no activity on this issue recently. Has the issue been fixed, or does it still require the community's attention? If you require support or are requesting an enhancement or feature then please create a topic on the Joplin forum. This issue may be closed if no further activity occurs. You may comment on the issue and I will leave it open. Thank you for your contributions.

@github-actions github-actions bot added the stale An issue that hasn't been active for a while... label Sep 16, 2024
@mrjo118
Copy link
Author

mrjo118 commented Sep 16, 2024

Still an issue

@github-actions github-actions bot removed the stale An issue that hasn't been active for a while... label Sep 17, 2024
Copy link
Contributor

Hey there, it looks like there has been no activity on this issue recently. Has the issue been fixed, or does it still require the community's attention? If you require support or are requesting an enhancement or feature then please create a topic on the Joplin forum. This issue may be closed if no further activity occurs. You may comment on the issue and I will leave it open. Thank you for your contributions.

@github-actions github-actions bot added the stale An issue that hasn't been active for a while... label Oct 17, 2024
@mrjo118
Copy link
Author

mrjo118 commented Oct 17, 2024

Still an issue

@github-actions github-actions bot removed the stale An issue that hasn't been active for a while... label Oct 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug It's a bug
Projects
None yet
1 participant