Make WordPress Core

Opened 16 years ago

Closed 15 years ago

#15654 closed defect (bug) (fixed)

Admin bar menu widths

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

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 16 years ago.
15654.fix.opera.the.new.ie6.patch (483 bytes ) - added by ocean90 16 years ago.

Download all attachments as: .zip

Change History (12)

#1 @filosofo
16 years ago

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

#2 @nacin
16 years ago

  • Milestone Awaiting Review3.1
  • Owner set to koopersmith
  • Status newassigned

Going to move the width to a min-width.

@koopersmith
16 years ago

#3 @koopersmith
16 years ago

  • Keywords has-patch added

#4 @westi
16 years ago

  • Resolutionfixed
  • Status assignedclosed

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

#5 @ocean90
16 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 16 years ago by ocean90 (previous) (diff)

#6 @ocean90
16 years ago

  • Resolution fixed
  • Status closedreopened

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

#7 @nacin
16 years ago

  • Resolutionfixed
  • Status reopenedclosed

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

#8 follow-up: @johnbillion
15 years ago

  • Resolution fixed
  • Status closedreopened
  • Version3.3

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

#9 @johnbillion
15 years ago

  • Keywords has-patch removed

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

  • Resolutionfixed
  • Status reopenedclosed
  • Version 3.33.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.