Make WordPress Core

Opened 11 years ago

Closed 11 years ago

#26509 closed defect (bug) (fixed)

Cropped characters in post title

Reported by: sergeybiryukov's profile SergeyBiryukov Owned by: iammattthomas's profile iammattthomas
Milestone: 3.8 Priority: normal
Severity: major Version: 3.8
Component: Administration Keywords: has-patch commit
Focuses: ui Cc:

Description

First noticed the issue in Press This, but it affects the regular Edit Post screen as well.

There's a rule for < 782px that increases padding for #titlediv #title:
trunk/src/wp-admin/css/wp-admin.css?rev=26827#L11882

It crops some Russian characters (see the screenshot).

Introduced in [26134].

Attachments (3)

26509.png (11.1 KB) - added by SergeyBiryukov 11 years ago.
26509.patch (394 bytes) - added by SergeyBiryukov 11 years ago.
26509.after.png (11.1 KB) - added by SergeyBiryukov 11 years ago.

Download all attachments as: .zip

Change History (10)

@SergeyBiryukov
11 years ago

#1 @SergeyBiryukov
11 years ago

There's a rule for < 782px that increases padding for #titlediv #title

Actually, that rule doesn't change anything visually (aside from cropping characters).

#2 @SergeyBiryukov
11 years ago

Also, the characters don't have to be Russian :) Try entering "g" or "q".

#3 @azaozz
11 years ago

Styling text input fields doesn't work the same in all browsers. As far as I see top and bottom padding doesn't make any difference in Chrome, FF and IE on Win7. Leaving it at 3px 8px or even setting it to 0 8px looks fine in both "standard" and "responsive" modes.

#4 @iammattthomas
11 years ago

26509.patch looks fine to me; it doesn't seem to hurt anything to eliminate that padding.

#5 @nacin
11 years ago

  • Keywords commit added

#6 @nacin
11 years ago

  • Severity changed from normal to major

#7 @iammattthomas
11 years ago

  • Owner set to iammattthomas
  • Resolution set to fixed
  • Status changed from new to closed

In 26830:

Eliminate the padding on #titlediv #title to prevent characters with descenders getting cropped in some browsers. Fixes #26509, props SergeyBiryukov.

Note: See TracTickets for help on using tickets.