Skip to content

Conversation

@lumpy72006
Copy link

Copy link
Member

@rgaudin rgaudin 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 ;

First of all, the UI doesn't look right: field is not placed correctly.

Image

Also, when hovering the toggle button, it disapears, which is annoying because most of the time, you want to see what you type(d) but keep your pointer on the icon to easily revert to hidden. screenshot doesn't show the pointer but it's on the icon

Image

Finally, it would be better if this could be more reusable (probably useful elsewhere like on the SSID)

  • store the SVGs in files
  • work of classes and match elements relatively
  • also place the JS part outside body block (there's a javascript block for that.

@rgaudin
Copy link
Member

rgaudin commented Jan 21, 2026

Convention is also I think to have the icon inside the field. If inline SVG is easier, you can make it reusable by creating a template for the complete field.

@lumpy72006
Copy link
Author

Thank you ;

First of all, the UI doesn't look right: field is not placed correctly.

Image Also, when hovering the toggle button, it disapears, which is annoying because most of the time, you want to see what you type(d) but keep your pointer on the icon to easily revert to hidden. screenshot doesn't show the pointer but it's on the icon Image

Could you tell me your OS + browser and whether you're in dark mode or not? This is what mine looks like and the toggle button doesn't disappear on hover (I'm on debian bookworm)
image

@rgaudin
Copy link
Member

rgaudin commented Jan 21, 2026

Firefox 147 on macOS. You should check your browser's console as the style is not applied properly: not using font, button style is off, colors as well.

@lumpy72006 lumpy72006 requested a review from rgaudin January 21, 2026 14:32
@lumpy72006
Copy link
Author

Firefox 147 on macOS. You should check your browser's console as the style is not applied properly: not using font, button style is off, colors as well.

I added CDNs to raw_base.html and it showed me what you had albeit the colors being off. This is what I have now after working on it
image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Allow unmasking of password when entering it

2 participants