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

[Feature Request] Allow customization of the placeholder field for a password input in dynamic parameters #2971

Open
Lordmau5 opened this issue Jan 14, 2025 · 0 comments
Labels
Enhancement A feature request or improvement

Comments

@Lordmau5
Copy link

Lordmau5 commented Jan 14, 2025

Describe the solution you'd like
The https://github.com/crowbartools/Firebot/blob/master/src/gui/app/directives/controls/dynamic-parameter.js#L37 password field / option should support letting the script developer set a custom placeholder value similar to how it's done for the number and searchable select inputs.

Additional context
This would allow using a password field for e.g. API keys while both preventing a potential leak when screensharing to users (either on stream or for technical help) while also setting the placeholder to for example Enter API key instead of Enter password, hopefully preventing confusion for the end user as well ("Do I have to put in my login password or my API key into this field?")

Extra info:
Perhaps this could also be considered for the string input for similar reasons? (More clarity to the user on what to enter into the field)
https://github.com/crowbartools/Firebot/blob/master/src/gui/app/directives/controls/dynamic-parameter.js#L32-L33

@Lordmau5 Lordmau5 added the Enhancement A feature request or improvement label Jan 14, 2025
@github-project-automation github-project-automation bot moved this to Todo in Firebot Jan 14, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Enhancement A feature request or improvement
Projects
Status: Todo
Development

No branches or pull requests

1 participant