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

Disable placeholder for password input #16128

Merged
merged 2 commits into from Apr 7, 2024
Merged

Conversation

Alanhou1222
Copy link
Contributor

References

Fixes #15977

Code changes

Setting line history placeholder conditionally on whether options.password is set for input widget

User-facing changes

Users will not see line history placeholder text in a password input field after the change

Backwards-incompatible changes

None

Copy link

Thanks for making a pull request to jupyterlab!
To try out this branch on binder, follow this link: Binder

@krassowski krassowski added this to the 4.1.x milestone Apr 7, 2024
@krassowski krassowski changed the title disable placeholder for password input object Disable placeholder for password input Apr 7, 2024
Copy link
Member

@krassowski krassowski left a comment

Choose a reason for hiding this comment

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

Thank you @Alanhou1222!

@krassowski krassowski merged commit 2a8257a into jupyterlab:main Apr 7, 2024
80 of 81 checks passed
Copy link

welcome bot commented Apr 7, 2024

Congrats on your first merged pull request in this project! 🎉
congrats
Thank you for contributing, we are very proud of you! ❤️

@krassowski
Copy link
Member

@meeseeksdev please backport to 4.1.x

meeseeksmachine pushed a commit to meeseeksmachine/jupyterlab that referenced this pull request Apr 7, 2024
krassowski pushed a commit that referenced this pull request Apr 7, 2024
Co-authored-by: Alan Hou <alanhou911222@gmail.com>
gderocher pushed a commit to gderocher/jupyterlab that referenced this pull request Apr 26, 2024
* disable placeholder for password input object

* Remove unnecessary translation for empty string

---------

Co-authored-by: Michał Krassowski <5832902+krassowski@users.noreply.github.com>
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.

JupyterLab puts confusing prompt in input field for input and getpass
2 participants