Make WordPress Core

Opened 4 years ago

Closed 4 years ago

Last modified 4 years ago

#19139 closed defect (bug) (fixed)

"View Site" and other items below site/admin title need to grow

Reported by: nacin Owned by: koopersmith
Milestone: 3.4 Priority: normal
Severity: normal Version: 3.3
Component: Toolbar Keywords: has-patch
Focuses: Cc:

Description (last modified by nacin)

See curious appearance in 18197#comment:108.

Attachments (1)

admin-bar-submenu-width.diff (334 bytes) - added by jakemgold 4 years ago.
Sets min-width to 100% on submenus in admin bar

Download all attachments as: .zip

Change History (17)

#1 @nacin
4 years ago

  • Description modified (diff)

#2 @jakemgold
4 years ago

  • Owner set to jakemgold
  • Status changed from new to accepted

#3 @jakemgold
4 years ago

Setting min-width to 100% on the submenu UL appears to fix this... tested in Chrome 15, Firefox 7, and IE7, IE8, IE9 with good results.

Patch following this comment...

4 years ago

Sets min-width to 100% on submenus in admin bar

#4 @jakemgold
4 years ago

  • Keywords has-patch added

#5 @nacin
4 years ago

It would be nice if "Visit Site" can shrink as well (if it's an only child), that way it doesn't look as weird. see #19139.

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

#6 @nacin
4 years ago

Sorry, I meant, see #19184.

#7 @ryan
4 years ago

  • Component changed from UI to Admin Bar

#8 @nacin
4 years ago

  • Owner changed from jakemgold to koopersmith
  • Status changed from accepted to assigned

koop for review.

#9 @koopersmith
4 years ago

  • Status changed from assigned to accepted

This patch doesn't play nicely with top-level secondary menus — .quicklinks ul targets all menus, not just submenus.

It should be easily fixed, though. Looking into it now.

#10 @ryan
4 years ago

  • Version set to 3.3

#11 @azaozz
4 years ago

  • Resolution set to fixed
  • Status changed from accepted to closed

Don't see this any more, seems fixed in the meantime.

#12 @ryan
4 years ago

  • Milestone changed from 3.3 to Future Release
  • Resolution fixed deleted
  • Status changed from closed to reopened

Still exists if you have a long title. We can live with it for 3.3. Reopening and punting to future.

#13 @duck_
4 years ago

Closed #19491 as dupe. Has alternative patch.

#14 @camiloclc
4 years ago

What about just setting min-width on a direct child selector? Like ".ab-top-menu > .menupop > .ab-sub-wrapper"?

#15 @koopersmith
4 years ago

  • Resolution set to fixed
  • Status changed from reopened to closed

In [19717]:

Allow toolbar submenus to expand to the width of the parent item. Props camiloclc, fixes #19139.

#16 @SergeyBiryukov
4 years ago

  • Milestone changed from Future Release to 3.4
Note: See TracTickets for help on using tickets.