Skip to content

feat(login): add password visibility and clean up block titles#20

Merged
max-wells merged 1 commit intorust-ui:mainfrom
krishpranav:fix/login-password-ux-and-block-titles
Apr 12, 2026
Merged

feat(login): add password visibility and clean up block titles#20
max-wells merged 1 commit intorust-ui:mainfrom
krishpranav:fix/login-password-ux-and-block-titles

Conversation

@krishpranav
Copy link
Copy Markdown
Contributor

@krishpranav krishpranav commented Apr 10, 2026

Description

Login blocks looked polished, but they were still missing a few practical details users expect in real auth forms.
This patch closes that gap and keeps block naming aligned with what each block actually renders.

What changed

  • Added password visibility toggle support to login01, login02, login03, and login04
  • Added autocomplete="username" on login email inputs
  • Added autocomplete="current-password" and minlength=8 on password inputs
  • Extended shared Input API to support autocomplete and minlength
  • Renamed login and sidenav block titles to be accurate and consistent in the registry
  • Added tests to lock title mappings and password input expectations

Validation

cargo test -p app --lib block_entry -- --nocapture
running 3 tests
test domain::blocks::block_entry::tests::login_block_titles_are_accurate_and_consistent ... ok
test domain::blocks::block_entry::tests::sidenav_block_titles_are_accurate_and_consistent ... ok
test domain::blocks::block_entry::tests::login_blocks_include_password_field_security_and_usability_features ... ok

test result: ok. 3 passed; 0 failed; 0 ignored; 0 measured; 0 filtered out; finished in 0.00s

Screenshot

Image

Closes #10

The login blocks now behave more like real forms: email/password fields include proper autocomplete hints, password inputs enforce a minimum length, and each layout gets a show/hide password toggle.\n\nI also updated the login and sidenav block titles so the registry labels match what the blocks actually render, then added regression tests in block_entry to keep those mappings (and the password field expectations) from drifting.
Copy link
Copy Markdown
Contributor

@max-wells max-wells left a comment

Choose a reason for hiding this comment

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

Great contribution — clean implementation, tests included, and good attention to UX details (autocomplete, minlength, password toggle). Happy to merge this in. Thanks @krishpranav!

@max-wells max-wells merged commit e97a72f into rust-ui:main Apr 12, 2026
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.

Blocks section.. Small fixes

2 participants