Skip to content

Password reveal toggle - #31

Open
lainefox wants to merge 2 commits into
pgaskin:masterfrom
lainefox:password-view-toggle
Open

lainefox wants to merge 2 commits into
pgaskin:masterfrom
lainefox:password-view-toggle

Conversation

@lainefox

@lainefox lainefox commented Nov 1, 2026

Copy link
Copy Markdown

this pull request adds a button to reveal the password in the auth dialog. useful for typing in long or complex passwords.

Screenshot_2026-11-01-03-12-42-05_f91e775ae319b8b37fa711ccfa25d824

changes:

  • adds a icon toggle button in the field with an eye icon, generally a standard feature across android apps (54fb73e)
  • problem found during first test iteration: clicking the button shifted the position on the screen of the dialog, also causing the hitboxes of the buttons to desync with their actual location (d4bd3a5)

testing:

  • on my personal phone (oneplus nord) by myself and also by opencode sending touch events through adb
  • on the android studio emulator (pixel 9)

changes partially assisted by a llm (opencode, model: big pickle)

Enable TextInputLayout's password toggle end icon mode for password
fields in prompt dialogs so users can inspect and verify entered
passwords.
SessionActivity uses shortEdges cutout mode for immersive fullscreen
display. Dialog overlays inherited this from the activity's theme,
allowing WindowManager to center the dialog into the top cutout and
status bar area when the soft keyboard opened.

On any subsequent relayout request (such as toggling password visibility
with transformation methods), WindowManager clamped the window's decor
frame below the cutout without updating the underlying window input
geometry, causing the visual dialog to jump down by the status bar
height while touch hitboxes stayed at the previous coordinates.

Setting windowLayoutInDisplayCutoutMode to never on the dialog overlay
ensures dialogs are placed below the cutout immediately when the IME
opens.

This branch has not been deployed

No deployments
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.

2 participants