Make WordPress Core

Opened 17 years ago

Closed 17 years ago

#5832 closed defect (bug) (fixed)

Write -> Post formatted badly on small screens

Reported by: mdawaffe's profile mdawaffe Owned by:
Milestone: 2.5 Priority: normal
Severity: normal Version: 2.5
Component: Administration Keywords: has-patch
Focuses: Cc:

Description

Currently, we have .wrap { margin-right: 15% }. This rule makes the Write screen look bad on small monitors.

Attached are two images. One shows the current state on a 800px wide monitor, the other shows what it would look like if the right margin matched the left (15px).

Is there a minimum resolution we're going to support?

Attachments (4)

WordPress write screen - big right margin.png (116.9 KB) - added by mdawaffe 17 years ago.
current
WordPress write screen - thin right margin.png (116.6 KB) - added by mdawaffe 17 years ago.
right margin matches left (15px)
css-margin.diff (1.4 KB) - added by jhodgdon 17 years ago.
Patch that changes right margins on admin screens to 15px instead of 15%
5832.diff (1013 bytes) - added by iammattthomas 17 years ago.
Patch sets right margin to fixed width, reduces page min-width to 770px.

Download all attachments as: .zip

Change History (16)

@mdawaffeEmeritus Committer
17 years ago

current

@mdawaffeEmeritus Committer
17 years ago

right margin matches left (15px)

#1 @ionfish
17 years ago

I must admit to being somewhat baffled by that margin too. I don't have an especially tiny monitor (1280x800), but I don't have my browser window anywhere close to maximised, so the huge gap on the right of the admin pages in WordPress squashes what's actually quite a densely packed set of functionality into a rather small space.

#2 @lloydbudd
17 years ago

  • Severity changed from minor to normal
  • Version set to 2.5

#3 @jhodgdon
17 years ago

+1 on changing the right margin. I'm attaching a patch that changes it on all the admin screens. I tested the patch in Firefox 2.0 and IE 6 (yes, I still have IE6 around for testing purposes), with a variety of browser window widths.

(By the way, the admin menu looks downright terrible in IE6, and the dashboard doesn't show any panels; I realize that is a separate issue, is someone planning to fix it? I personally don't care much, but it does make WP look bad.)

@jhodgdon
17 years ago

Patch that changes right margins on admin screens to 15px instead of 15%

#4 @jhodgdon
17 years ago

  • Keywords has-patch added

#5 @ryanLead Tester
17 years ago

Work is being done on TinyMCE to allow its toolbar to wrap.

#6 @DD32
17 years ago

+1

The right margin affects all other pages as well, Not just the write pages, While it'd be good for TinyMCE to wrap its toolbars, The fact theres a fair bit of rel-estate going unused still needs to be fixed for the rest of us.

On my larger(21") monitor, It leaves a ~200px wide blank area, Which looks drastically out of place compared with the much smaller left margin.

#7 @iammattthomasEmeritus Committer
17 years ago

+1. I'm in complete agreement with the comments mentioned above. I'm submitting a new patch that includes the min-width specified for the benefit of TinyMCE. With these changes, the admin fits into a 770px wide window.

@iammattthomasEmeritus Committer
17 years ago

Patch sets right margin to fixed width, reduces page min-width to 770px.

#8 @ryanLead Tester
17 years ago

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

(In [7031]) Adjust margin and sidemenu. Props mt. fixes #5832 #6003

#9 follow-ups: @DD32
17 years ago

  • Resolution fixed deleted
  • Status changed from closed to reopened

The Notices class needs updating too by the looks.

View /wp-admin/plugins.php?deactivate=true and the bar will only stretch accross to about that of the users tab.

#10 in reply to: ↑ 9 @DD32
17 years ago

Replying to DD32:

The Notices class needs updating too by the looks.

And by that, I was of course refering to this:

<div id="message" class="updated fade"><p>Plugin <strong>deactivated</strong>.</p></div>

#11 in reply to: ↑ 9 @iammattthomasEmeritus Committer
17 years ago

Replying to DD32:

The Notices class needs updating too by the looks.

Fixed in patch for #6019.

#12 @ryanLead Tester
17 years ago

  • Resolution set to fixed
  • Status changed from reopened to closed
Note: See TracTickets for help on using tickets.