-
Notifications
You must be signed in to change notification settings - Fork 602
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
[REQUEST] -password option in sript mode #1273
Comments
It seems like |
Please follow the guidelines! f.e. do not report unrelated bugs/issues onto other issues. |
@DaveDavenport hello! Is there any plans to add the future in the near future? |
no, given you can have multiple modi enabled and switch between them, there is no point to this. Either the pasword becomes visible when switching between modi, or you cannot see what you enter when you include this mode. |
As a developer, I only set the script mode for the rofi session, so the user cannot switch modes in my window. |
I am not sure how that is relevant for a generic feature? |
(A mode imho is a standalone thing that a user adds to his list of modi. How it is going to be called and combined with other modi is up to the user, not the developer). |
The server of my application starts rofi as |
So it is not a mode at all? Why not use dmenu emulation that is more targeted at this use? |
Because the script mode avoids flickering between the display of rofi windows |
dmenu has a patch to hide password |
@RaZ0rr-Two not sure what you are trying to say? rofi dmenu emulation also has a option to hide the password. |
Oh, my bad. I thought op was asking for an option to add hide password option. Didn't know it already came with it. All good |
Any update on this feature?. I would be really nice. Im trying to make a network manager UI. And i find rofi-dmenu to be a bit janky (since it flickers when the script is blocking), and rofi-script is easier to write than dmenu |
No. If there was an update, this issue would have mentioned it. |
Before creating a feature request
I checked the next branch to see if the feature has already been
implemented
I searched existing reports to see if it is already requested.
What is the user problem or growth opportunity you want to see solved?
Please add support of
-password
option(and other useful options) in script mode.How do you know that this problem exists today? Why is this important?
The following cases not hides user input:
'\0password\n'
.Who will benefit from it?
The world.
Version
1.6.1
Configuration
Default configuration
The text was updated successfully, but these errors were encountered: