[wp-trac] [WordPress Trac] #48894: Improve the small user sub window at the right corner on the admin bar a.k.a. the Howdy fly-out
WordPress Trac
noreply at wordpress.org
Fri Jul 3 09:18:33 UTC 2020
#48894: Improve the small user sub window at the right corner on the admin bar
a.k.a. the Howdy fly-out
-------------------------------------------------+-------------------------
Reporter: ixkaito | Owner: afercia
Type: enhancement | Status: assigned
Priority: normal | Milestone: 5.5
Component: Toolbar | Version:
Severity: normal | Resolution:
Keywords: has-patch has-screenshots needs- | Focuses: ui,
refresh | accessibility
-------------------------------------------------+-------------------------
Comment (by afercia):
@isabel_brison thanks for the patch! I see a few issues.
More importantly:
1. When using the Spacebar, we need to emulate the native behavior: **the
Spacebar key activates buttons when it is released**. That's why
[attachment:"48894.3.diff"] was using the `keyup` event instead of
`keydown`. That part shouldn't be changed.
2. Emulating a mobile device (do set "Mobile" in the Chrome dev tools) the
menus don't open any longer. This is the reason why the previous patch
removed the event listeners when `ontouchstart` is present. See comment:18
Minor:
- I see a missing `@param` notation on a function docblock.
- Running `grunt jshint`, it tells me there are 7 errors in the `admin-
bar.js` file.
--
Ticket URL: <https://core.trac.wordpress.org/ticket/48894#comment:49>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform
More information about the wp-trac
mailing list