Skip to content

feat: add password visibility toggle to password fields#6161

Open
akshitrawat21 wants to merge 3 commits intoFlowiseAI:mainfrom
akshitrawat21:feature/password-visibility-toggle
Open

feat: add password visibility toggle to password fields#6161
akshitrawat21 wants to merge 3 commits intoFlowiseAI:mainfrom
akshitrawat21:feature/password-visibility-toggle

Conversation

@akshitrawat21
Copy link
Copy Markdown

Summary

Closes #6160

  • Added eye icon toggle to Input component for all type='password' fields — covers Register page (Password & Confirm Password)
  • Added show/hide toggle to Old Password, New Password, and Confirm New Password fields in Account Settings → Security section

Changes

  • packages/ui/src/ui-component/input/Input.jsx — added showPassword state, IconEye/IconEyeOff as endAdornment when inputParam.type === 'password'
  • packages/ui/src/views/account/index.jsx — added showOldPassword, showNewPassword, showConfirmPassword states with eye icon toggles on each OutlinedInput

Test plan

  • Go to /register — verify eye icon appears on Password and Confirm Password fields
  • Click eye icon — password text becomes visible
  • Click again — password is masked
  • Log in and go to Account Settings → Security section
  • Verify eye icon on Old Password, New Password, Confirm New Password
  • Verify toggle works independently for each field
  • Verify no regression on other input fields (non-password types have no eye icon)

Screenshots

image image image

- Add eye icon toggle to Input component for all password type fields
- Add show/hide toggle to Old Password, New Password, Confirm New Password
  fields in Account Settings security section
- Closes FlowiseAI#6160
Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request introduces password visibility toggling functionality across the application. It updates the generic Input component and the account settings page to include eye icons that allow users to mask or unmask password fields. A review comment points out a redundant conditional check in the getInputType function that can be simplified.

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.

feat: Add password visibility toggle (eye icon) for password fields

1 participant