Skip to content

feat: add toggle for password field visibility#1252

Open
bntrtm wants to merge 1 commit into
stoatchat:mainfrom
bntrtm:main
Open

feat: add toggle for password field visibility#1252
bntrtm wants to merge 1 commit into
stoatchat:mainfrom
bntrtm:main

Conversation

@bntrtm

@bntrtm bntrtm commented Jun 3, 2026

Copy link
Copy Markdown

Implements a password toggle to TextField components configured as passwords.
The changes are implemented by simply returning the appropriate fieldConfiguration properties pertinent to password visibility toggling as outlined by the MDUI documentation.

The feature, widespread on the web today, greatly improves user login experience. Without it, users are disincentivized from using stronger passwords, as some may opt for a shorter and/or simpler one to avoid the potential headache of re-entering the password should a typo occur in the process.

How was this PR tested?

  • Ensured toggle hides password in TextField as expected
  • Ensured login page password field features the toggle
  • Ensured create page password field features the toggle

Screenshots & Screencasts (if appropriate)

These two screenshots show only the signup page, but it works all the same on the login page.

2026-06-03_stoat-password-toggle_OFF 2026-06-03_stoat-password-toggle_ON

Checklist:

  • I have carefully read the contributing guidelines
  • I have performed a self-review of my own code
  • I have made corresponding changes to the documentation if applicable
  • I have no unrelated changes in the PR
  • I have confirmed that any new dependencies are strictly necessary
  • I have written tests for new code (if applicable)
  • I have followed naming conventions/patterns in the surrounding code

Please declare, if any, LLM usage involved in creating this PR

None

Signed-off-by: bntrtm <git.bntrtm@tjbainter.com>
@bntrtm bntrtm requested review from a team as code owners June 3, 2026 20:11

@Pecacheu Pecacheu left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looks good to me, using a feature already built into mdui. (Tho just to note, I believe we aren't merging any more PRs atm waiting on a release candidate)

@Dadadah

Dadadah commented Jun 3, 2026

Copy link
Copy Markdown
Member

Thanks for your contribution. Chu is correct, we will merge this after 0.8.0 is released.

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.

3 participants