Opened 18 months ago

Closed 6 months ago

Last modified 6 months ago

#19254 closed defect (bug) (fixed)

Alignment of labels in "Add media" dialog

Reported by: waclawjacek Owned by: nacin
Priority: normal Milestone: 3.5
Component: I18N Version: 3.2.1
Severity: minor Keywords: has-patch
Cc:

Description

Please see the attached screenshot. It shows how the labels are aligned in the "From URL" tab (and not on the "From my computer" tab!) in the Polish translations of the "Add media" dialog. The bug is present in WP 3.3 beta 3.

It can be fixed by switching the order of <span class="alignleft"> and <label> in that form (that is how the "From my computer" form is written, plus a <br class="clear"> tag).

I'm attaching two patches to this report. The only change in them is the aforementioned switch in all places where I found <span class="alignleft"> preceding a <label>. One of the patches has the "<br class="clear">" tag along with the switch, the other doesn't. The patches should be applied to wp-admin/includes/media.php.

Take care,
WJ

Attachments (5)

patch-with-br.diff (2.4 KB) - added by waclawjacek 18 months ago.
With the <br class="clear" /> tags.
patch-without-br.diff (2.3 KB) - added by waclawjacek 18 months ago.
Without the <br class="clear" /> tags.
add-media-dialog.png (47.8 KB) - added by waclawjacek 18 months ago.
A screenshot of the bug.
19254.png (59.5 KB) - added by ocean90 8 months ago.
19254-refresh.diff (2.4 KB) - added by DrewAPicture 7 months ago.

Download all attachments as: .zip

Change History (15)

comment:1 follow-up: ↓ 2   nacin18 months ago

Is this present in 3.2?

With the <br class="clear" /> tags.

Without the <br class="clear" /> tags.

comment:2 in reply to: ↑ 1   waclawjacek18 months ago

Replying to nacin:

Is this present in 3.2?

Yes.

  • Version changed from 3.3 to 3.2.1

A screenshot of the bug.

  • Milestone changed from Awaiting Review to 3.5

Let's see how it looks with the new media workflow.

  • Keywords punt added

No need to punt it, if the screen will be removed.

  • Keywords needs-refresh has-patch added

I don't think the screen can be removed - it's called directly by plugins quite often. patch-without-br.diff appears sane enough, but needs to be made from root.

  • Keywords needs-refresh removed
  • Owner set to nacin
  • Resolution set to fixed
  • Status changed from new to closed

In 22444:

Put the label outside the span to ensure margins when the text wraps. props waclawjacek. fixes #19254.

  • Keywords punt removed
Note: See TracTickets for help on using tickets.