Make WordPress Core

Opened 11 years ago

Closed 11 years ago

#34040 closed defect (bug) (fixed)

Multisite - My Sites dropdown should have hover effect for parent menu items

Reported by: Nikschavan Owned by: afercia
Priority: normal Milestone: 4.4
Component: Toolbar Version: 4.4
Severity: normal Keywords: has-patch commit
Cc: Focuses: ui, accessibility, multisite

Description

In WordPress Multisite, When selecting sites from My Sites dropdown, the parent menu items are not highlighted.

http://i.imgur.com/NnVPp0u.jpg

Attachments (2)

34040.1.diff (636 bytes ) - added by Nikschavan 11 years ago.
Patch for the fix
34040.2.diff (1.9 KB ) - added by afercia 11 years ago.

Download all attachments as: .zip

Change History (14)

@Nikschavan
11 years ago

Patch for the fix

#1 @Nikschavan
11 years ago

  • Keywords has-patch added

#2 @Nikschavan
11 years ago

  • Summary My Sites dropdown in a Multisite should have hover effect for parent menu itemsMultisite - My Sites dropdown should have hover effect for parent menu items
  • Type defect (bug)enhancement

#4 @johnbillion
11 years ago

  • Keywords reporter-feedback needs-testing added

Which browser and OS are you using? This works fine for me in the latest Firefox and Chrome on OS X.

#5 @Nikschavan
11 years ago

  • Keywords reporter-feedback removed

@johnbillion The above screenshot is from latest chrome in windows 10, same is the result in latest FF.

Meanwhile, the fix is working on latest chrome and FF on windows 10 -

http://i.imgur.com/CdZH4vs.jpg

Will test the patch on more browsers if required.

#6 @Nikschavan
11 years ago

  • Keywords needs-testing removed

#7 @swissspidy
11 years ago

  • Component Networks and SitesToolbar
  • Milestone Awaiting Review4.4
  • Type enhancementdefect (bug)

I just tested this on Chrome and FF on OS X and can confirm the bug exists. The provided patch fixes the issue.

#8 @DrewAPicture
11 years ago

  • Owner set to afercia
  • Status newassigned

#9 @afercia
11 years ago

Thanks @Nikschavan! As far as I can see there are two more things we should take care of:

  • the icon color when hovering or focusing the sub items should have the "active" color
  • all the changes here should be done also for the Sass alternate color schemes, see /wp-admin/css/colors/_admin.scss

#10 @Nikschavan
11 years ago

Thank you for a review @afercia, I will refresh my patch with SASS and add hover for the icons as well!

@afercia
11 years ago

#11 @afercia
11 years ago

  • Focuses accessibility added
  • Keywords commit added

Refreshed patch to take care of the icon and color schemes. Screenshot with the patch applied:

https://cldup.com/xbYZsN2_kT.png

#12 @afercia
11 years ago

  • Resolutionfixed
  • Status assignedclosed

In 35265:

Multisite: Improve the Admin Bar parent menu items color on hover and focus.

Props Nikschavan for the initial patch.
Fixes #34040.

Note: See TracTickets for help on using tickets.