Make WordPress Core

Opened 10 years ago

Closed 10 years ago

#32242 closed enhancement (fixed)

Improve spacing of the Excerpt textarea.

Reported by: paulwilde's profile paulwilde Owned by: obenland's profile obenland
Milestone: 4.3 Priority: normal
Severity: normal Version: 4.3
Component: Posts, Post Types Keywords: has-patch
Focuses: ui, administration Cc:

Description

This patch does the following:

  • Removes the attachmentlinks class (I couldn't find a single instance of this in the codebase?)
  • Adds display: block to the textarea to remove the 3-4 pixels below the textarea. (Maybe all <textarea>s should be a block element?) - See here.
  • Makes the textarea 100% width.
  • Adds better spacing above the textarea.

https://s3.amazonaws.com/f.cl.ly/items/3B141l3V1K0j133r082b/excerpt-metabox.png

Attachments (1)

excerpt-metabox.diff (387 bytes) - added by paulwilde 10 years ago.

Download all attachments as: .zip

Change History (8)

#1 @paulwilde
10 years ago

  • Keywords has-patch added

#2 @SergeyBiryukov
10 years ago

  • Component changed from General to Posts, Post Types
  • Focuses administration added
  • Milestone changed from Awaiting Review to 4.3

#3 @obenland
10 years ago

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

#4 @obenland
10 years ago

.attachmentlinks was introduced in [3303]. I'm now trying to track down when that was corresponding HTML element was removed.

#5 @obenland
10 years ago

Removed in [25472], unused since [6910]. Since the deprecated function removed the markup, I think it's save to remove the CSS selector as well.

#6 @obenland
10 years ago

  • Owner changed from SergeyBiryukov to obenland
  • Status changed from assigned to accepted

#7 @obenland
10 years ago

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

In 32841:

More consistent spacing around the excerpt textarea.

Props paulwilde.
Fixes #32242.

Note: See TracTickets for help on using tickets.