Make WordPress Core

Opened 5 years ago

Last modified 2 years ago

#48222 assigned enhancement

"Show password" button overlaps with the LastPass icon

Reported by: sergeybiryukov's profile SergeyBiryukov Owned by:
Milestone: Future Release Priority: normal
Severity: normal Version: 5.3
Component: Login and Registration Keywords: has-screenshots has-patch needs-design-feedback early needs-refresh
Focuses: ui Cc:

Description

The new "Show password" button added to login screen in [46256] overlaps with the LastPass extension icon. Tested with Google Chrome 77 on Windows 10.

This only happens on Log In and Reset Password screens. The Edit User screen is OK, as the button there is separate from the input.

Attachments (29)

48222.login-screen.PNG (7.4 KB) - added by SergeyBiryukov 5 years ago.
48222.reset-password-screen.PNG (13.1 KB) - added by SergeyBiryukov 5 years ago.
48222.edit-user-screen-ok.PNG (7.6 KB) - added by SergeyBiryukov 5 years ago.
48222.diff (373 bytes) - added by sabernhardt 5 years ago.
moves button from the (right) edge when input has background-image in style
patch-login-show-password-button-lastpass.png (4.9 KB) - added by sabernhardt 5 years ago.
login form after patch
Sign in - Google Accounts 2019-10-06 07-08-00.jpg (61.8 KB) - added by adamsilverstein 5 years ago.
google sign in
Image 2019-10-06 at 7.09.53 AM.png (26.8 KB) - added by adamsilverstein 5 years ago.
Log In ‹ Develop WordPress Title — WordPress 2019-10-06 07-19-24.jpg (32.5 KB) - added by adamsilverstein 5 years ago.
login with data-lpignore="true" in password field
48222.2.diff (1.6 KB) - added by adamsilverstein 5 years ago.
data-lpignore="true" approach
48222.3.diff (368 bytes) - added by sabernhardt 5 years ago.
simplest option: only adding min-width to password input
login-simple-min-width-firefox.png (8.0 KB) - added by sabernhardt 5 years ago.
simplest option in Firefox (min-width)
48222.4.diff (1.1 KB) - added by sabernhardt 5 years ago.
"fake input" option with border on container and larger input areas for login
fake-input-lastpass-edge.png (10.5 KB) - added by sabernhardt 5 years ago.
"fake input" option in Edge, with LastPass icons
48222 Safari LastPass.png (47.7 KB) - added by afercia 5 years ago.
See the Safari "key" icon on the username field.
lastpass-chrome-macos-20200116.jpg (64.7 KB) - added by valentinbora 5 years ago.
Test of the login form with patch 48222.4.diff on Chrome/LastPass
Screen Shot 2020-02-07 at 9.17.22 AM.png (11.3 KB) - added by garrett-eclipse 5 years ago.
Move the eye outside of the input
48222-password-ideas.png (13.7 KB) - added by valentinbora 5 years ago.
Some more layout ideas so we have options to choose from
Screen Shot 2020-02-17 at 9.58.10 PM.png (25.9 KB) - added by valentinbora 5 years ago.
Desktop alignment and sizing of latest design with button in line with Password label
Screen Shot 2020-02-17 at 10.02.29 PM.png (28.3 KB) - added by valentinbora 5 years ago.
Reset password screen alignment and sizing of latest design with button in line with Password label
48222.5.diff (3.5 KB) - added by valentinbora 5 years ago.
Implementation redone based on design feedback, button above field in line with label
48222 focus style.png (23.7 KB) - added by afercia 5 years ago.
Focus style from 48222.5.diff​
48222.6.png (52.8 KB) - added by valentinbora 5 years ago.
Screenshots of login and reset password forms with show/hide button underneath input control
48222.6.diff (4.5 KB) - added by valentinbora 5 years ago.
Show/hide password button underneath input control with full length label
48222.7.diff (3.0 KB) - added by valentinbora 4 years ago.
Patch refresh
48222.8.diff (3.7 KB) - added by sabernhardt 3 years ago.
restores button focus style, removes redundant aria-label, adjusts margin and padding
main-login-lastpass-patch-8.png (7.9 KB) - added by sabernhardt 3 years ago.
focus outline and margin with 48222.8.diff (Firefox/Windows, LastPass)
hide-password.png (14.7 KB) - added by poena 3 years ago.
Hide password button on focus
48222.9.diff (3.9 KB) - added by sabernhardt 3 years ago.
moving button between strength indicator and weak password confirmation for reset-password form
48222.10.diff (3.9 KB) - added by peterwilsoncc 3 years ago.

Download all attachments as: .zip

Change History (100)

#1 @SergeyBiryukov
5 years ago

  • Focuses ui added

@sabernhardt
5 years ago

moves button from the (right) edge when input has background-image in style

@sabernhardt
5 years ago

login form after patch

#2 @sabernhardt
5 years ago

The patch checks for a background image in the password input's style attribute. If it finds one, the show password button moves over by 40 pixels.

#3 @johnbillion
5 years ago

How do services that use the "show" button such as Facebook, Google, and WordPress.com handle this?

#4 @adamsilverstein
5 years ago

Attaching some screenshots of the Google account login screen.

Here, the show password icon looks like it is in the right hand edge of the input - in fact, its outside the input entirely and uses a merged borderless edge to make it appear as part of the field.

#5 @adamsilverstein
5 years ago

One other potential option would be to disable the lastpass icon for the password field by adding data-lpignore="true". I verified this works and that I can still fill in both fields using the icon visible in the user_name field.

@adamsilverstein
5 years ago

login with data-lpignore="true" in password field

@adamsilverstein
5 years ago

data-lpignore="true" approach

#6 @sabernhardt
5 years ago

I tested the second patch in Edge, IE11, Opera 63, Firefox 69 and Chrome 77 on Windows 10. I had no trouble populating the two fields with LastPass when clicking the icon inside the Username/Email input or when using the show/hide button. It looks cleaner than my option, and I'm not sure having a clickable LastPass icon in the Password field is necessary after the previous field.

#7 @Clorith
5 years ago

LastPass being just one of many password managers, I would be more inclined to take the route of the other big players out there; a "fake" part of the input with no left border as noted in comment:4 would be more forward-compatible as well with whatever new services may pop up.

#8 @garrett-eclipse
5 years ago

I would also agree with the fake portion of the input. OR moving the icon up beside the 'Password' label so it's not directly in the input itself.

#9 @davidbaumwald
5 years ago

  • Keywords dev-feedback added

I agree with the "faking" of an extended input as well, instead of chasing an unknown number of vendor-specific workarounds.

@sabernhardt
5 years ago

simplest option: only adding min-width to password input

@sabernhardt
5 years ago

simplest option in Firefox (min-width)

@sabernhardt
5 years ago

"fake input" option with border on container and larger input areas for login

@sabernhardt
5 years ago

"fake input" option in Edge, with LastPass icons

#10 @sabernhardt
5 years ago

  • Keywords has-patch needs-testing added

I included a simpler option of only the min-width because that should not require much testing.

For the "fake input" option, I tested in various PC browsers as well as at different zoom levels and with a larger text size. That still leaves plenty of other testing (including High Contrast Mode).

This ticket was mentioned in Slack in #accessibility by sabernhardt. View the logs.


5 years ago

#12 @afercia
5 years ago

For the records, this was noted also on the Show Password ticket. See:
https://core.trac.wordpress.org/ticket/42888#comment:24
https://core.trac.wordpress.org/ticket/42888#comment:45

That said, I'm not sure WordPress should necessarily remediate to poor implementations by external software. It appears to me Lastpass should make a better job and take padding into consideration. I'd lean towards reporting upstream.

#13 @davidbaumwald
5 years ago

  • Milestone changed from 5.3 to Future Release

With 5.3 RC1 releasing today, this is being moved to Future Release. If any committer feels this can be worked in quickly for 5.3 or can assume ownership in 5.4, feel free to move it back up.

#14 @johnbillion
5 years ago

  • Milestone changed from Future Release to 5.3.1

#15 @afercia
5 years ago

@johnbillion what are your thoughts on reporting upstream to LastPass? Seems to me LastPass should do a better job and its icon placement seems arguable. It also conflicts with native browsers UI. See for example Safari in th screenshot below. I think this also applies to Edge.

@afercia
5 years ago

See the Safari "key" icon on the username field.

#16 @johnbillion
5 years ago

Observation: google.com and facebook.com both place the view button outside of the input (with a fake focus border). wordpress.com uses the padding method that core is using.

This ticket was mentioned in Slack in #forums by sterndata. View the logs.


5 years ago

This ticket was mentioned in Slack in #core by audrasjb. View the logs.


5 years ago

This ticket was mentioned in Slack in #core by audrasjb. View the logs.


5 years ago

#21 @audrasjb
5 years ago

  • Keywords needs-design needs-design-feedback added
  • Milestone changed from 5.3.1 to 5.4

Hi,

48222.4.diff is a nice workaround, but I'm wondering about implementing a fake input since focus outline looks very strange to me… and this will be the rendering for all users, even those who don't use LastPass or so.

Adding needs-design/needs-design-feedback workflow keyword and moving it to 5.4 as it doesn't sounds realistic for 5.3.1 which is planned this week.

Last edited 5 years ago by audrasjb (previous) (diff)

#22 @sabernhardt
5 years ago

Yes, the fake input option could be better, so I agree to reconsider this for a later version. And it looks stranger in High Contrast Mode. Wrapping the username input with a similar container div might be a good start to improving it, though I'm not sure how yet.

Another consideration: the icon for Norton's Password Manager also overlaps.

@valentinbora
5 years ago

Test of the login form with patch 48222.4.diff on Chrome/LastPass

#23 @fahimmurshed
5 years ago

Hi,

I have checked that, this issue is not solved yet.
https://i.imgur.com/fHtA9Jf.png

I have modified this file /wp-admin/css/login.min.css

Change the padding

    padding: 0px 45px;

CSS code:

.login .button.wp-hide-pw {
    background: 0 0;
    border: 1px solid transparent;
    box-shadow: none;
    font-size: 14px;
    line-height: 2;
    width: 2.5rem;
    height: 2.5rem;
    min-width: 40px;
    min-height: 40px;
    margin: 0;
    padding: 0px 45px;
    position: absolute;
    right: 0;
    top: 0;
}

It should look like this
https://i.imgur.com/SyJR41t.png

#24 @xkon
5 years ago

Usually any additional icons are added as a background image from extensions from what I've seen in general.

In a similar way to one of @sabernhardt early patches and the later of @fahimmurshed , instead of moving the eye icon we can force any background image to a different position.

As an example an addition like this to login.css would push the LastPass icon further on the left side instead.:

.login input#user_pass {
	background-position: 85% !important;
}

This is a simple solution imho that doesn't change the current input in any way and also avoids the confusion of the focused element not being in "full width".

Last edited 5 years ago by xkon (previous) (diff)

#25 @sabernhardt
5 years ago

@fahimmurshed and @xkon: I like the idea of a simpler option. Unfortunately, moving the LastPass background image does not change the clickable area's position (behind the show/hide eye button). And the clickable area would be covered by the button with an increased padding, too.

I considered a right margin instead of padding, but then the eye button would cover the long passwords WordPress suggests (the first patch here has the same problem).

#26 @afercia
5 years ago

It's a bit unfortunate LastPass doesn't add anything to reliably target with CSS the fields it applies a background image to. the only option would be something like:

.login input[type="password"][style^="background-image"] { ... }

which is a bit crazy. Still thinking that LastPass should do a better job and the issue should be fixed upstream.

@garrett-eclipse
5 years ago

Move the eye outside of the input

#27 @garrett-eclipse
5 years ago

Random thought, could we just move the eye outside of the input to avoid the conflict all together?
https://core.trac.wordpress.org/raw-attachment/ticket/48222/Screen%20Shot%202020-02-07%20at%209.17.22%20AM.png

@valentinbora
5 years ago

Some more layout ideas so we have options to choose from

#28 follow-up: @karmatosed
5 years ago

Of any of these, the 2nd one would be my preference showing the eye and 'show password' underneath.

#29 @fahimmurshed
5 years ago

Both are awesome
https://i.imgur.com/P6GvV7o.png

#30 in reply to: ↑ 28 @afercia
5 years ago

Replying to karmatosed:

Of any of these, the 2nd one would be my preference showing the eye and 'show password' underneath.

I'd agree. I'd just consider to make the clickable area a bit bigger.

@karmatosed would you prefer to further discuss with the design team or can the needs-design and needs-design-feedback keywords be removed?

#31 @karmatosed
5 years ago

  • Keywords needs-design removed

I would like to see it in a PR before totally looping design out of this. Let's remove the needs-design and keep design feedback keyword.

#32 @valentinbora
5 years ago

@karmatosed looking back at it after a couple of days rest, does the link need to have such high contrast (i.e. be so prominent)? I'd personally vote for turning it down a notch. Thoughts?

This ticket was mentioned in Slack in #design by karmatosed. View the logs.


5 years ago

#34 @nrqsnchz
5 years ago

I think that @valentinbora's mockups are great. I particularly like these two (option 1 and 2):

https://cldup.com/cfboSj_iwr.png

The label "Show password" is more obvious, easier to understand than just the icon.

I'm also more inclined to the first version, where the label is above the field. In my opinion it is easier to see.

looking back at it after a couple of days rest, does the link need to have such high contrast

@valentinbora I think the contrast is fine.

#35 @karmatosed
5 years ago

  • Keywords needs-design-feedback removed

#36 @karmatosed
5 years ago

I agree the contrast is fine for me. I think it would be great to get this into trunk and see what we think there. Thanks, everyone for the great work involved so far.

#37 @paaljoachim
5 years ago

I think the above option 1 in Enrique's post where the show password (I think both words need to begin with capital letters) is on the same line as Password bring better symmetry and balance in the overall image. The eye moves along the top border line from Password over to Show Password.

Option 2. Brings a stronger contrast where the eye needs to move from the Password text then down right across the password box to the show password.

Option 1 is easier on the eyes.

@valentinbora
5 years ago

Desktop alignment and sizing of latest design with button in line with Password label

@valentinbora
5 years ago

Reset password screen alignment and sizing of latest design with button in line with Password label

@valentinbora
5 years ago

Implementation redone based on design feedback, button above field in line with label

#38 follow-up: @Clorith
5 years ago

I'm not a fan of the markup here, to reach the button to toggle visibility one would then have to navigate backwards, and would also be exposed to a potentially contextless "Show" before even being presented with the input.

The toggle button needs its old label of Show password, we don't know what direction the user reaches the button, so retaining the context within the label is important.

The button should also be moved to come after the input form, you can still use CSS for positioning, but markup needs to follow a natural pattern.

#39 @valentinbora
5 years ago

@Clorith there seems to be some divergence between your feedback and #design (@melchoyce recommended the shorter label due to potential i18n constraints with longer verbiage in some languages). Please refer to https://wordpress.slack.com/archives/C02S78ZAL/p1581966078468900

Also, @nrqsnchz commented (see Slack link above) re. a11y for the button to be right after the label for context.

@Clorith please help out for reaching consensus here, then we can tune the code accordingly.

#40 in reply to: ↑ 38 @afercia
5 years ago

  • Keywords needs-refresh added; dev-feedback needs-testing removed

Replying to Clorith:

I'm not a fan of the markup here, to reach the button to toggle visibility one would then have to navigate backwards, and would also be exposed to a potentially contextless "Show" before even being presented with the input.

I agree with @Clorith.
UI controls need to make sense also when read out of context. A contextless "Show" would be the equivalent of a "Read more" link that doesn't specify the "what".

Re: the order: many users experience the content in a linearized way. Keyboard users, screen reader users, etc. The order of elements needs to be logical: first the "what", and then the actions related to the "what" object.

The option 2 on comment:34 is the only one that satisfies these needs.

Also, looking at 48222.5.diff :

  • there's a missing semicolon in wp-login.php at line 968 which throws a PHPCS error: <?php _e( 'Hide' ) ?>
  • the CSS class button-secondary is deprecated and can be removed: it's a pre-existing issue but since we're here... See [38672].

#41 @afercia
5 years ago

Looking a bit more in depth:

  • as mentioned in a previous comment, the clickable area is a bit too small: we're down from the previous 40 by 40 pixels to 50 by 20 pixels: the height is too small especially for touch on small screens, not to mention accessibility
  • the focus style could be improved a bit (see screenshot below)
  • as there's now visible text, there's no need to have an aria-label: it should be removed together with the JS part meant to update its value
  • some CSS values seem a bit "magic numbers", for example line-height: 2.15; based on font-size: 12px; computes to 25.8 pixels: it would be nice to have some meaningful numbers instead (with no decimals)
  • on button activation, the icon updates but the visible text doesn't and it's always "Show"
Last edited 5 years ago by afercia (previous) (diff)

@afercia
5 years ago

Focus style from 48222.5.diff​

#42 @valentinbora
5 years ago

Alright, thanks for chiming in.

Next steps:

  1. Move button below input control
  2. Revert to full label to include the noun "password" (translated length will no longer be an issue due to new location)
  3. Update text to Show or Hide based on state via JS
  4. Increase button height
  5. Code style fixes

@valentinbora
5 years ago

Screenshots of login and reset password forms with show/hide button underneath input control

@valentinbora
5 years ago

Show/hide password button underneath input control with full length label

#43 @valentinbora
5 years ago

@afercia @Clorith patch updated per your request.

Please review and share feedback. Thanks!

#44 @audrasjb
5 years ago

  • Milestone changed from 5.4 to Future Release

Hi,

With 5.4 Beta 3 approaching and the Beta period reserved for bugs introduced during the cycle, this is being moved to Future Release. If any maintainer or committer feels this should be included or wishes to assume ownership during a specific cycle, feel free to update the milestone accordingly.

#45 @bobeaston
4 years ago

Have we forgotten this one?

#46 @indysigner
4 years ago

Hi all,

The WooCommerce login fields seem to have the same issue (as described at the top of this tread).

Here’s an example with the WC login fields:
https://www.drweb.de/mein-konto/

Wondering what’d be the best way to move forward from here?

Thank you,

Markus

Version 0, edited 4 years ago by indysigner (next)

@valentinbora
4 years ago

Patch refresh

#47 @valentinbora
4 years ago

@audrasjb how do we get attention back onto this one, hoping it could make it into a release?

#48 @audrasjb
4 years ago

  • Keywords needs-refresh removed
  • Milestone changed from Future Release to 5.8

Thanks for bringing attention to that one. Moving for 5.8 consideration :)

@sabernhardt
3 years ago

restores button focus style, removes redundant aria-label, adjusts margin and padding

@sabernhardt
3 years ago

focus outline and margin with 48222.8.diff (Firefox/Windows, LastPass)

#49 @sabernhardt
3 years ago

  • Keywords needs-testing added

Changes in 48222.8.diff:

  • The password visibility button needs to keep the focus outline (the change in color is insufficient).
  • A small amount of padding (2px) is on both the left and right sides of the button for better readability with the focus outline.
  • The button has additional margin on the top and bottom so it does not touch either the input or the submit button. The click target itself is still 30 pixels tall, though the spacing around it adds 16 more.
  • The aria-label is removed because the button text gives that information already.
  • The password input does not have the side padding anymore, which had made room for the button in front of it.

#50 @sabernhardt
3 years ago

  • Milestone changed from 5.8 to 5.9
  • Owner set to sabernhardt
  • Status changed from new to accepted

I can claim ownership for this in 5.9.

@poena
3 years ago

Hide password button on focus

#51 @poena
3 years ago

  • Keywords commit added; needs-testing removed

The patch works well. Added a screenshot of the focus style that shows the padding that was described above.

#52 @sabernhardt
3 years ago

  • Keywords needs-refresh added; commit removed

Thanks for the updated screenshot! I tried with the reset password form, though, and that apparently needs positioning adjustments. (The password strength indicator div might need to move before the button.)

@sabernhardt
3 years ago

moving button between strength indicator and weak password confirmation for reset-password form

#53 @sabernhardt
3 years ago

  • Keywords needs-testing added; needs-refresh removed

This ticket was mentioned in Slack in #core by sabernhardt. View the logs.


3 years ago

This ticket was mentioned in Slack in #core-test by hellofromtonya. View the logs.


3 years ago

This ticket was mentioned in Slack in #core by audrasjb. View the logs.


3 years ago

This ticket was mentioned in Slack in #core-test by boniu91. View the logs.


3 years ago

#58 @mai21
3 years ago

  • Keywords needs-design-feedback added

Test Report

Env

  • WordPress 5.9-alpha-52170 (reproducible on login page while having other strange view for reset pw page)
  • Chrome Version 95.0.4638.54 on Ubuntu, while 93.0.4577.82 for Windows
  • Windows 10 and Ubuntu 18.04.6
  • Theme: Twenty Twenty-One
  • Plugin: WP Downgrade , WP beta tester, WooCommerce 5.9

Steps to test

  1. Add lastPass chrome extension
  2. Open the login page for wordpress
  3. Show/hide pw overlap with lastpass icon

With the latest patch https://core.trac.wordpress.org/attachment/ticket/48222/48222.9.diff, the show/hide is outside pw field so no overlap now but please check the following notes:
1- Clicking show/hide pw , the btn width will change with click https://jmp.sh/uwsbY87
2- Spacing above/below show /hide pw is too small with respect to the rest of the page
3- PW itself will overlap with LastPass icon https://jmp.sh/k5aGpGz
4- For reset PW, it was as follow https://jmp.sh/77tDJ6C

Note: I believe this ticket needs design team

#59 @sabernhardt
3 years ago

  • Keywords early added
  • Milestone changed from 5.9 to 6.0

Let's revisit this for next release, earlier in the cycle.

#60 @peterwilsoncc
3 years ago

48222.10.diff is 48222.9.diff but refreshed against trunk to remove offsets, clean up line ending types.

@sabernhardt do you need anything other than design feedback to take this forward?

While testing, the screenshots @poena uploaded a few months ago appear to still be representative.

#61 @sabernhardt
3 years ago

That screenshot looks the same (to me) as 48222.10.diff, but this patch does need more than design feedback.

  • Setting display: block on .login .button.wp-hide-pw overrides .no-js .hide-if-no-js.
  • The JS change affects the user pages (Profile, New User, Edit User). The text includes "password" on larger screens, and the ARIA label is stuck at "Hide password" whether the password is showing or not.

Regarding the design:

  • Preventing overlap of password manager icons over the password itself seems impossible without including extra padding. That would not make sense for people who do not use a password manager.
  • I do not want to try forcing a width for the toggle button (for example, using both text strings in span tags and hiding one with visibility: hidden). However, moving the icon to the other side of the text could make the width difference between the two text strings less noticeable.
  • The bottom margin could be slightly more (16px instead of 12).

I drafted a PR to explore some revisions. (Note: The PR is still on my fork and may change.)

This ticket was mentioned in PR #2466 on WordPress/wordpress-develop by sabernhardt.


3 years ago
#62

Make room for password managers such as LastPass by moving the visibility toggle button, and show the button label text.

Trac ticket: https://core.trac.wordpress.org/ticket/48222

Moving the eye icon after the text can make the width difference less noticeable. Plus, this adds a little more to the bottom margin.

Login

https://i0.wp.com/user-images.githubusercontent.com/17100257/160030018-329ab195-451c-4bad-9553-8493d70de788.jpg

Reset Password

https://i0.wp.com/user-images.githubusercontent.com/17100257/160030019-6935bb79-71a3-41f0-81cb-23864bc5bc25.jpg

This ticket was mentioned in Slack in #design by peterwilsoncc. View the logs.


3 years ago

sabernhardt commented on PR #2466:


3 years ago
#65

### Screenshots in Arabic (RTL), _without_ a Password Manager

Login

https://i0.wp.com/user-images.githubusercontent.com/17100257/160658968-0fc6364f-5493-4360-8e0f-916340a8d860.png

Reset Password

https://i0.wp.com/user-images.githubusercontent.com/17100257/160658977-d262b126-9dc4-44af-8d86-12e52203cc3b.png

This ticket was mentioned in Slack in #design by chaion07. View the logs.


3 years ago

This ticket was mentioned in Slack in #core by sabernhardt. View the logs.


3 years ago

#68 @sabernhardt
3 years ago

  • Keywords needs-refresh added; needs-testing removed
  • Type changed from defect (bug) to enhancement

The patch needs to reflect the new autocomplete values, and would need further revisions to make the login toggle button consistent with the profile page and potential new uses on setup-config.php and options-writing.php (#3534 and #9883).

Changing the type to enhancement since this is well beyond the original scope of a bugfix.

This ticket was mentioned in Slack in #core by chaion07. View the logs.


3 years ago

#70 @chaion07
3 years ago

  • Milestone changed from 6.0 to Future Release

We reviewed this ticket during a recent bug-scrub. Based on the feedback from the team we feel that this ticket including #3534 & #9883 can be moved to Future Release considering only the few days remaining before Feature Freeze gets into effect from 12 April 2022.

Props to @sabernhardt

Cheers!

#71 @sabernhardt
2 years ago

  • Owner sabernhardt deleted
  • Status changed from accepted to assigned

If someone wants to work on a patch that makes the visibility buttons consistent in all places, feel free to take ownership.

In case showing a separate border around the button and the input is acceptable, I tried using flex in this PR. The PR is on my fork because it does not follow the direction this ticket has taken, but I can move it to the main repo if it's worth fixing the bug sooner.

Note: See TracTickets for help on using tickets.