Switch button icons to show effect, not status

[dan-jensen]

Description

This switches the button icons so they indicate the effect they will
have if clicked, instead of the current status. This will make it more
obvious that the button is a button, and what effect it will have if clicked.

Motivation and Context

Clickable buttons generally indicate the effect they will have if
clicked. For example, if a video is playing there is usually a pause button to indicate the video will become paused if the pause button is clicked.

Previously, the buttons in this plugin looked like
status indicators instead of buttons. When the password was hidden by default,
the button showed the "not visible" icon. That meant it was not necessarily obvious
the button was a button.

How Has This Been Tested?

Currently running in a production environment, see https://www.pocketero.com/users/open-account

Screenshots (if appropriate)

Types of changes

• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to not work as expected)
• Non-breaking change (aesthetic only)

Checklist

• My code follows the code style of this project.

Gratitude

@coliff thanks for creating this plugin, it works great! 👍

[coliff]

thanks for this. well spotted. :-) This now matches the UI that Microsoft IE11 and Edge use.
Tested using simple <input type="password" /> test-case: https://input-type-password.stackblitz.io/