[wp-trac] [WordPress Trac] #43241: Twenty Seventeen: Menu behaviour is not proper

[WordPress Trac]

#43241: Twenty Seventeen: Menu behaviour is not proper
-----------------------------+------------------------------
 Reporter:  nilamacharya     |       Owner:
     Type:  defect (bug)     |      Status:  new
 Priority:  normal           |   Milestone:  Awaiting Review
Component:  Bundled Theme    |     Version:  4.7
 Severity:  normal           |  Resolution:
 Keywords:  has-screenshots  |     Focuses:  ui
-----------------------------+------------------------------
Changes (by afercia):

 * keywords:   => has-screenshots
 * version:  4.9.4 => 4.7


Comment:

 Yes, the `focus` CSS class is added also for keyboard navigation, without
 that, the menu doesn't work when using only the keyboard. Worth noting the
 menu stays "sticky" only with links with an href attribute `#` or links
 with `target="_blank"` regardless if they're external or internal links.
 The reason is: focus stays on the clicked menu item link.

 I'm not even sure this should be fixed, as there's no reason to use links
 to `#` and maybe it's acceptable to see the menu still open when closing a
 browser tab previously opened by a link with `target="_blank"`. In any
 case, I'd recomment to not use `blur()` as it would cause a focus loss.

--
Ticket URL: <https://core.trac.wordpress.org/ticket/43241#comment:4>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform