Skip to content

Fix: improve password toggle tooltip behavior#130

Open
Apekshaa007 wants to merge 3 commits into
Canopus-Labs:mainfrom
Apekshaa007:fix-password-toggle-tooltip
Open

Fix: improve password toggle tooltip behavior#130
Apekshaa007 wants to merge 3 commits into
Canopus-Labs:mainfrom
Apekshaa007:fix-password-toggle-tooltip

Conversation

@Apekshaa007

Copy link
Copy Markdown
Contributor

Overview

This PR improves the user experience of the password input field by refining the password visibility toggle and its tooltip behavior in the Input.jsx component. The update ensures clearer visual feedback and more consistent interaction when users toggle password visibility.

Changes Implemented

  1. Improved password visibility toggle logic in Input.jsx
  2. Fixed tooltip behavior for the password show/hide icon
  3. Ensured consistent UI feedback during hover/focus states
  4. Enhanced input interaction flow for better usability

Purpose of this Change

The previous implementation had inconsistencies in tooltip display and interaction clarity for the password visibility toggle. This update ensures users receive proper visual guidance while switching between hidden and visible password states.

Testing Done

  1. Verified password toggle works correctly (show/hide functionality)
  2. Checked tooltip visibility on hover/focus states
  3. Tested input behavior across different browsers
  4. Ensured no regression in normal input fields

Impact

Improved UX for authentication-related input fields
Better accessibility and user feedback
No breaking changes introduced

Related Issue

Closes #99

@KaranUnique

Copy link
Copy Markdown
Contributor

@Apekshaa007 The Input.jsx change looks good. Could you remove the unrelated package-lock.json changes from this PR to keep the scope focused?

@Apekshaa007

Copy link
Copy Markdown
Contributor Author

Hii @KaranUnique . I have removed the unrelated changes and i am really sorry for the delay from my side. i was busy with exams. I hope now it could be merged.

@KaranUnique

Copy link
Copy Markdown
Contributor

@Apekshaa007 I checked the PR, but the only file modified appears to be package-lock.json. I couldn't find any changes related to the password toggle tooltip behavior mentioned in the PR title.

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.

BUG: Add Tooltip/Text for Password Visibility Toggle Icon

2 participants