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

invert password on-off icons #2008

Merged
merged 1 commit into from
May 11, 2019
Merged

invert password on-off icons #2008

merged 1 commit into from
May 11, 2019

Conversation

drhaliburton
Copy link
Contributor

Password on/off toggle icon showing the wrong icon for the hide/show password state.

When the password is hidden the icon should have the eye with the line through it, when the password is visible/plain text, the icon should be the eye with no line.

Steps to reproduce:

  • Create md-field component with password input field
  • Toggle will show eye with no line for hidden password
  • Hit password toggle, eye with line through will show for visible password

Vue-material => https://i.imgur.com/H3YkZqV.png (current bug)
Material.io => https://i.imgur.com/chsuQY3.png (correct usage)

@marcosmoura marcosmoura merged commit 3e9dfa2 into vuematerial:dev May 11, 2019
marcosmoura added a commit that referenced this pull request May 12, 2019
* origin:
  docs: fix 2 wrong property names with autocomplete docs (#2006)
  fix(MdTable): undefined table sorting #1906 (#2012)
  docs: fix dynamic tooltip text to show bottom instead of top (#2013)
  refactor(MdDatepicker): upgrade date-fns (#2037)
  fix(MdDatepicker): positioning when used inside flex container (#2036)
  fix(MdSelect): infinite loop error, when no v-model provided (#2035)
  fix(MdApp): normalized component's tag before checking to match slot … (#1994)
  chore: some misspellings in config.js (#1986)
  fixFixed non unique ids on MdSelect (#2001)
  fix(MdInput): invert password on-off icons (#2008)
  test: fix test selectors for checkbox and switch (#2009)
  docs: add default theme as a suggestion (#2043)
  Fixed md-wave transition (#2034)
  fix(MdRipple): fixed missing ripple effect (#2059)
marcosmoura added a commit that referenced this pull request May 12, 2019
* origin: (80 commits)
  docs: remove wrong text-shadow
  docs: add referal code
  fix(MdProgressSpinner): improve transition (#1946)
  chore: upgrade deps
  docs: fix 2 wrong property names with autocomplete docs (#2006)
  fix(MdTable): undefined table sorting #1906 (#2012)
  docs: fix dynamic tooltip text to show bottom instead of top (#2013)
  refactor(MdDatepicker): upgrade date-fns (#2037)
  fix(MdDatepicker): positioning when used inside flex container (#2036)
  fix(MdSelect): infinite loop error, when no v-model provided (#2035)
  fix(MdApp): normalized component's tag before checking to match slot … (#1994)
  chore: some misspellings in config.js (#1986)
  fixFixed non unique ids on MdSelect (#2001)
  fix(MdInput): invert password on-off icons (#2008)
  test: fix test selectors for checkbox and switch (#2009)
  docs: add default theme as a suggestion (#2043)
  Fixed md-wave transition (#2034)
  fix(MdRipple): fixed missing ripple effect (#2059)
  docs: add codefund ads and remove old sponsors
  fix(MdSvgLoader): svg loader for invalid / missing mimetype (#1942)
  ...
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.

2 participants