Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feature(#1005): provide a click handler for click on right icon for K… #1027

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

RueRivoli
Copy link
Collaborator

This MR aims at creating a click handler on KtField right icons so that KtFieldPassword can hide/show password when the user clicks on the eye icon.

@github-actions github-actions bot added package:kotti-ui @3yourmind/kotti-ui scope:documentation Affects the documentation labels Nov 27, 2024
@Isokaeder
Copy link
Collaborator

I am wondering about why this feature affects all kotti fields, since this feature should only affect KtFieldPassword according to the issue #1005

Wouldn't it be easier to add a boolean field to only KtFieldPassword that lets you opt in (or better opt out) of this toggle. Something like:

<!-- Will have the eye icon to toggle the obscure/reveal password -->
<KtFieldPassword />

<!-- Will not have that toggle -->
<KtFieldPassword hideVisibilityToggle />

@RueRivoli RueRivoli force-pushed the 1005-reveal-hide-passwords-option branch from c3c940e to 5f37620 Compare November 28, 2024 17:55
@github-actions github-actions bot added the package:yoco @3yourmind/yoco label Nov 28, 2024
…tField

This is needed for KtPassword to reveal/hide passwords to user
@RueRivoli RueRivoli force-pushed the 1005-reveal-hide-passwords-option branch from 5f37620 to a1191a0 Compare December 3, 2024 17:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
package:kotti-ui @3yourmind/kotti-ui package:yoco @3yourmind/yoco scope:documentation Affects the documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants