Make WordPress Core

Opened 10 years ago

Closed 7 years ago

#29076 closed defect (bug) (wontfix)

The Minor Publishing Actions (Buttons) need some breathing room

Reported by: nikv's profile NikV Owned by:
Milestone: Priority: normal
Severity: normal Version: 3.8
Component: Posts, Post Types Keywords: has-patch ui-feedback commit
Focuses: ui, administration Cc:

Description

Both the Save Draft and Preview Buttons may look good in their current state above Status, Visibility, etc. But when you remove the misc. publishing actions (like it is done in bbPress) the Minor Publishing Actions tend to look a bit odd. The buttons come out odd when you use #misc-publishing-actions and display: none. The main place I see this is in bbPress's create new form editor. I also think just in general it would be nice to just have a bit more white space around the minor publishing actions. I am also not sure if this may work as a WordPress bug. Perhaps I should take it to the bbPress trac?

I have attached an image of how it looks like in bbPress. I have also attached the two diff files that change it.

Attachments (3)

Screen Shot 2014-07-31 at 11.16.54 AM.png (12.9 KB) - added by NikV 10 years ago.
common.css.diff (327 bytes) - added by NikV 10 years ago.
common-rtl.css.diff (338 bytes) - added by NikV 10 years ago.

Download all attachments as: .zip

Change History (11)

@NikV
10 years ago

@NikV
10 years ago

#1 @SergeyBiryukov
10 years ago

  • Component changed from General to Editor
  • Focuses ui added

#2 @NikV
10 years ago

  • Keywords has-patch ui-feedback 2nd-opinion added

#3 @celloexpressions
10 years ago

  • Version changed from trunk to 3.8

#4 @iseulde
9 years ago

  • Component changed from Editor to Posts, Post Types

#5 @FolioVision
8 years ago

  • Keywords 2nd-opinion removed

Tested the patch, works as advertised.

#6 @karmatosed
7 years ago

  • Keywords commit added
  • Milestone changed from Awaiting Review to Future Release

I'd maybe explore adding more padding to the bottom to balance it out @NikV. I agree this is a good idea but maybe something more like:

https://cldup.com/S9Cya081L9.png

It would be good to combine both the rtl and non rtl into one patch, for ease.

This ticket was mentioned in Slack in #design by karmatosed. View the logs.


7 years ago

#8 @karmatosed
7 years ago

  • Milestone Future Release deleted
  • Resolution set to wontfix
  • Status changed from new to closed

As work is going on with Gutenberg that will make this not an issue, closing for now. We discussed this in this weeks design triage meeting.

Note: See TracTickets for help on using tickets.