carbon-website icon indicating copy to clipboard operation
carbon-website copied to clipboard

[PasswordInput live demo]: Password input eye icon placement is off

Open dannybranton opened this issue 3 years ago • 2 comments

Summary

I noticed the Password input eye show/hide icon placement moved for the react version of the component. Could the same change be added to the vanilla js component? I assume this is so password manager icons aren't obstructed.

Newer version / react image

Older version / vanilla image

Justification

No response

Desired UX and success metrics

No response

Required functionality

No response

Specific timeline issues / requests

No response

Available extra resources

No response

Code of Conduct

dannybranton avatar Oct 12 '21 19:10 dannybranton

Well, I clicked the React storybook and it looks like its the older way there too

https://react.carbondesignsystem.com/?path=/story/components-textinput--toggle-password-visibility

I saw the possibly new version on the live demo at https://www.carbondesignsystem.com/components/text-input/usage/, but perhaps its a bug instead

dannybranton avatar Oct 12 '21 19:10 dannybranton

@dannybranton The password visibility icon should still be on the input. I think the icon change you're seeing on the website live demo is a bug, or it may not be up to date.

jnm2377 avatar Oct 20 '21 16:10 jnm2377

Replaced with storybook demo https://github.com/carbon-design-system/carbon-website/pull/3716

alisonjoseph avatar Sep 08 '23 15:09 alisonjoseph