Make WordPress Core

Opened 10 years ago

Closed 10 years ago

#33212 closed defect (bug) (fixed)

Add some more space to german and norwegian (bokmål) quick edit labels

Reported by: clorith's profile Clorith Owned by: helen's profile helen
Milestone: 4.3 Priority: low
Severity: trivial Version: 4.3
Component: Quick/Bulk Edit Keywords: has-patch commit dev-reviewed
Focuses: ui Cc:

Description

German (formal) does not align properly in the 4.3 trunk when using quick edit.

Attached patch fixes this, and also fixes the Norwegian (Bokmål) issue for the same area.

Attachments (14)

33212.patch (1.0 KB) - added by Clorith 10 years ago.
33212-before.png (62.3 KB) - added by ocean90 10 years ago.
33212-before-mobile.png (78.6 KB) - added by ocean90 10 years ago.
33212-after.png (60.0 KB) - added by ocean90 10 years ago.
33212-after-mobile.png (82.1 KB) - added by ocean90 10 years ago.
33212.2.patch (2.0 KB) - added by ocean90 10 years ago.
Screen Shot 2015-07-31 at 16.28.24 1.png (35.4 KB) - added by iseulde 10 years ago.
33212.3.patch (3.8 KB) - added by SergeyBiryukov 10 years ago.
33212.4.patch (2.8 KB) - added by ocean90 10 years ago.
33212-bulk-edit-ru_RU.png (14.8 KB) - added by SergeyBiryukov 10 years ago.
33212-ru_RU-after-5.png (110.1 KB) - added by ocean90 10 years ago.
33212.5.patch (3.0 KB) - added by ocean90 10 years ago.
33212.6.patch (3.0 KB) - added by ocean90 10 years ago.
33212.7.patch (3.5 KB) - added by ocean90 10 years ago.

Download all attachments as: .zip

Change History (38)

@Clorith
10 years ago

#1 @Clorith
10 years ago

  • Keywords has-patch added

#2 @SergeyBiryukov
10 years ago

  • Component changed from General to Quick/Bulk Edit
  • Milestone changed from Awaiting Review to 4.3
  • Summary changed from Add some more space to german (formal) and norwegina (bokmål) quick edit labels to Add some more space to german (formal) and norwegian (bokmål) quick edit labels

Previously: #25753

#3 @ocean90
10 years ago

  • Summary changed from Add some more space to german (formal) and norwegian (bokmål) quick edit labels to Add some more space to german and norwegian (bokmål) quick edit labels

It's needed for both variants of German.
Before and after screenshots would be great.

#4 @transl8or
10 years ago

Screenshots of what?
I just figured that it looks all fine in 4.2.3 in (normal) german, formal german und swiss german (which is just different in die ß (german esszet) vs the swiss version uses 'ss').

And now in 4.3 RC1 normal and formal german is out of the grid.
And swiss german has now the english date format, meaning month dropdown before the day input field.

@ocean90
10 years ago

@ocean90
10 years ago

@ocean90
10 years ago

#6 @ocean90
10 years ago

  • Priority changed from normal to low
  • Severity changed from normal to trivial
  • Type changed from enhancement to defect (bug)

33212.2.patch is 33212.1.patch plus the same fix for ru_RU and resets the left margin for narrow screens. Before/after screenshots attached.

Version 0, edited 10 years ago by ocean90 (next)

#7 @iseulde
10 years ago

This is needed for Dutch too.

#8 @ocean90
10 years ago

Should we improve the default value? Maybe 6 or 6.5em?

#9 @iseulde
10 years ago

@ocean90 Probably the sanest solution. :)

#10 @iseulde
10 years ago

This is made to fit exactly for English... As soon as "Password" is a character longer it wraps.

#11 @SergeyBiryukov
10 years ago

Introduced in [32945], which changed the Date field from <label> to <legend>.

33212.2.patch does not fix the issue for me:

  • The width should be set for span.title, not for legend.
  • .timestamp-wrap needs to be adjusted in the same way as span.input-text-wrap.

See 33212.3.patch.

@ocean90
10 years ago

#12 follow-up: @ocean90
10 years ago

33212.3.patch breaks .timestamp-wrap for other languages, see https://cloudup.com/cmc0a1TC3gm. Fixed in 33212.4.patch.

33212.4.patch changes the default value to 6em and removes the extra styles for de_DE, de_DE_formal, and ru_RU. lt_LT is still needed because their translation of "slug" is "Nuoroda (slug)".

#13 in reply to: ↑ 12 ; follow-up: @SergeyBiryukov
10 years ago

Replying to ocean90:

33212.4.patch changes the default value to 6em and removes the extra styles for de_DE, de_DE_formal, and ru_RU.

6em is not enough for ru_RU, see 33212-bulk-edit-ru_RU.png.

7em would work.

#14 @pavelevap
10 years ago

Nice, 33212.4.patch fixes also cs_CZ (long "Slug" label translation).

"Date" label in mobile view should be above fields (as all other fields)?

#15 in reply to: ↑ 13 @ocean90
10 years ago

Replying to SergeyBiryukov:

6em is not enough for ru_RU, see 33212-bulk-edit-ru_RU.png.

7em would work.

That's the reason why we should always add screenshots for visual changes. Your screenshot is from Quick Edit for pages which has different/longer labels. I'm fine with adding 7em for ru_RU again.

#16 @obenland
10 years ago

  • Keywords needs-refresh added

@SergeyBiryukov would you mind updating the patch? It would be nice to have that fixed in RC2.

@ocean90
10 years ago

#17 @ocean90
10 years ago

  • Keywords needs-refresh removed

I already had 33212.5.patch on my disk.

This ticket was mentioned in Slack in #core by obenland. View the logs.


10 years ago

#19 @helen
10 years ago

  • Keywords needs-patch added; has-patch removed

Patch breaks "Date" label.

http://s.hyhs.me/c90s/image.png

@ocean90
10 years ago

@ocean90
10 years ago

#20 @ocean90
10 years ago

  • Keywords has-patch added; needs-patch removed

33212.7.patch works in Firefox (comment:19] and fixes responsiveness (comment:14).

#21 @SergeyBiryukov
10 years ago

  • Keywords commit added

33212.7.patch looks good.

In 33212.3.patch I changed the selector for consistency with label span.title, but it doesn't seem worth it.

#22 @obenland
10 years ago

  • Owner set to helen
  • Status changed from new to assigned

#23 @nacin
10 years ago

  • Keywords dev-reviewed added

Looks good to me.

#24 @ocean90
10 years ago

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

In 33598:

List tables: Adjust width of Quick Edit labels for longer translations.

props Clorith, SergeyBiryukov, ocean90.
fixes #33212.

Note: See TracTickets for help on using tickets.