Make WordPress Core

Opened 14 years ago

Closed 13 years ago

#15654 closed defect (bug) (fixed)

Admin bar menu widths

Reported by: johnbillion's profile johnbillion Owned by: koopersmith's profile koopersmith
Milestone: 3.1 Priority: normal
Severity: normal Version: 3.1
Component: Administration Keywords:
Focuses: Cc:

Description

r16301 added a fixed width of 160px to admin bar menus.

My User Switching plugin adds a menu item with the title "Switch back to [display_name] ([user_login])".

The fixed width menu is too narrow for this title in 99% of cases and the text gets cut off.

Can we give menu items a min-width instead of a width?

Attachments (2)

15654.diff (1.8 KB) - added by koopersmith 14 years ago.
15654.fix.opera.the.new.ie6.patch (483 bytes) - added by ocean90 14 years ago.

Download all attachments as: .zip

Change History (12)

#1 @filosofo
14 years ago

A better-looking solution might be to allow multiple-line elements. See #15611

#2 @nacin
14 years ago

  • Milestone changed from Awaiting Review to 3.1
  • Owner set to koopersmith
  • Status changed from new to assigned

Going to move the width to a min-width.

@koopersmith
14 years ago

#3 @koopersmith
14 years ago

  • Keywords has-patch added

#4 @westi
14 years ago

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

(In [17295]) Switch to using min-width so long titles can be displayed instead of using elipsis. Fixes #15654 props koopersmith.

#5 @ocean90
14 years ago

r17295 makes some odd things in Opera 11 (Mac and Windows), see http://grab.by/8orF. min-width will be ignored.

If I remove overflow: hidden from #wpadminbar .quicklinks .menupop ul li a it works.

Last edited 14 years ago by ocean90 (previous) (diff)

#6 @ocean90
14 years ago

  • Resolution fixed deleted
  • Status changed from closed to reopened

15654.fix.opera.the.new.ie6.patch tested in Chrome, Firefox, Safari and IE7/8/9. No problems.

#7 @nacin
14 years ago

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

(In [17317]) Remove cruft overflow:hidden from the admin bar items. props ocean90, fixes #15654.

#8 follow-up: @johnbillion
13 years ago

  • Resolution fixed deleted
  • Status changed from closed to reopened
  • Version set to 3.3

This has regressed with changes made as part of #18197.

#9 @johnbillion
13 years ago

  • Keywords has-patch removed

#10 in reply to: ↑ 8 @westi
13 years ago

  • Resolution set to fixed
  • Status changed from reopened to closed
  • Version changed from 3.3 to 3.1

Replying to johnbillion:

This has regressed with changes made as part of #18197.

Please open a new ticket for the regression as this was fixed in a previous release.

Note: See TracTickets for help on using tickets.