Make WordPress Core

Opened 13 years ago

Closed 13 years ago

Last modified 13 years ago

#18138 closed defect (bug) (fixed)

DFW editor area is 12px narrower than theme content width

Reported by: helenyhou's profile helenyhou Owned by: azaozz's profile azaozz
Milestone: 3.3 Priority: low
Severity: trivial Version: 3.2
Component: General Keywords:
Focuses: Cc:

Description

In the full screen editor, the default width ends up being 12px narrower than the theme content width. There is an inline style (JS-produced?) that is the content width plus 10px, but then there is padding of 10px on each side plus a 1px border on #wp-fullscreen-container. Thus the editor area is actually 12px too narrow.

I reproduced this using Twenty Eleven, Twenty Ten, and a custom theme. It's not always noticeable, but there is the occasional discrepancy in line breaks.

Attachments (2)

18138.diff (558 bytes) - added by helenyhou 13 years ago.
dfw-width-twentyeleven.png (132.3 KB) - added by helenyhou 13 years ago.

Download all attachments as: .zip

Change History (8)

#1 @nacin
13 years ago

  • Milestone changed from Awaiting Review to 3.2.2

Nice catch and thanks for verifying it with other themes. Good bug report.

@helenyhou
13 years ago

#2 @helenyhou
13 years ago

  • Keywords has-patch added

I was wrong about it being a JS-produced inline style, but I think this should do it.

#3 @azaozz
13 years ago

  • Keywords reporter-feedback added; has-patch removed
  • Milestone 3.2.2 deleted
  • Owner set to azaozz
  • Priority changed from normal to low
  • Severity changed from normal to trivial
  • Status changed from new to reviewing

While working on DFW I actually looked through a few themes and compared the width of the content (text) on the front end to the reported theme width. All had a bit of padding/margin so the actual width is few pixels smaller. This is reflected in DFW.

@helenyhou could you measure the actual content width on the site and compare that to the DFW width.

#4 @helenyhou
13 years ago

  • Keywords reporter-feedback removed

Sure - in Twenty Eleven, the content area is 584px wide at the largest, which is reflected in the regular visual editor. However, when in DFW, the content area is 572px instead. I checked this in Firebug - will attach a screenshot of a particularly noticeable example (ignore the content!). As previously stated, I noticed the same thing in Twenty Ten (628 instead of 640) and a custom theme (698 instead of 710).

I first noticed this when what should have just been a full-width image caused a horizontal scrollbar. Seems like it'd be better for DFW to be a little wider than needed as opposed to narrower, since the editor style could further constrain it horizontally, but that is my very very humble opinion.

#5 @azaozz
13 years ago

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

In [18666]:

Make DFW content width match exactly the reported width from the theme, props helenyhou, fixes #18138

#6 @azaozz
13 years ago

  • Milestone set to 3.3
Note: See TracTickets for help on using tickets.