Make WordPress Core

Opened 6 years ago

Closed 6 years ago

#10242 closed defect (bug) (fixed)

Media menu does not handle multiple lines

Reported by: xibe Owned by: azaozz
Milestone: 3.0 Priority: normal
Severity: normal Version: 2.8
Component: UI Keywords: has-patch tested
Focuses: Cc:


This bug has been reported on the fr_FR forum back in Feburary, and it is still true for 2.8.

Some plugin add another item in the Media menu. While this is perfectly acceptable, some translation might already use the full width for the three default items (see first attached capture), and therefore any additional item would be pushed to a second menu line.

That should be no problem, but in effect that second menu line pushes the first one off-screen, leaving only the second one to be seen (see second attached capture).

The translation has since been reduced, but the issue remains: the menu should be able to handle numerous sub-lines, just because the use might want to use many item-adding plugins...

Attachments (3)

wp28-mediamenu-fr.png (49.7 KB) - added by xibe 6 years ago.
overlong fr_FR Media menu
wp28-mediamenuNextgen.png (45.2 KB) - added by xibe 6 years ago.
same fr_FR Media menu, with NetxtGEN Gallery plugin activated
10242.patch (504 bytes) - added by edward mindreantre 6 years ago.

Download all attachments as: .zip

Change History (11)

6 years ago

overlong fr_FR Media menu

6 years ago

same fr_FR Media menu, with NetxtGEN Gallery plugin activated

#1 @xibe
6 years ago

  • Keywords needs-patch 2nd-opininon added

#2 @ryan
6 years ago

  • Component changed from Menus to UI
  • Owner changed from ryan to azaozz
  • Status changed from new to assigned

#3 @azaozz
6 years ago

  • Milestone changed from 2.9 to 3.0

No patch, moving to 3.0

#4 @edward mindreantre
6 years ago

  • Keywords has-patch added; needs-patch 2nd-opininon removed

Vive la France!

Here's a patch. I converted the ul#sidemenu to a normal, non-floated UL. It looks like normal but can now handle very long lines. It don't look too pretty when it breaks up to two rows but them's the breaks, eh?

#5 @xibe
6 years ago

Thanks Edward!

My cache doesn't seem to want to take it into account, but a quick test through FireBug shows that it's the right direction. It'd need a bit of spit'n'polish to make sure it stays preeety, but apart from that, thanks a lot :)

#6 @dd32
6 years ago

  • Keywords needs-testing added

#7 @ocean90
6 years ago

  • Keywords tested added; needs-testing removed

Works fine: http://grab.by/45hN

You can test it with

function test_tabs ($tabs) {
	$newtab = array(
	'1' => ('1 Gallery'),
	'2' => ('2 Gallery'),
	'3' => ('3 Gallery'),
	'4' => ('4 Gallery'),
	'5' => ('5 Gallery'),
	'6' => ('6 Gallery'),
    return array_merge($tabs,$newtab);
add_filter('media_upload_tabs', 'test_tabs');

#8 @markjaquith
6 years ago

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

(In [14661]) Update media upload window tabs CSS to support wrapping of tabs when they exceed the available width. props edward mindreantre. fixes #10242

Note: See TracTickets for help on using tickets.