WordPress.org

Make WordPress Core

Opened 7 years ago

Closed 7 years ago

#25426 closed defect (bug) (fixed)

Twenty Fourteen: Reduce post widget margin

Reported by: kraftbj Owned by: lancewillett
Milestone: 3.8 Priority: normal
Severity: normal Version:
Component: Bundled Theme Keywords: has-patch
Focuses: Cc:

Description

The widgets margin-bottom is currently at 48px which seems excessive when viewing on a screen (see screenshot), at least for the sidebar.

Attachments (4)

Screen Shot 2013-09-27 at 10.28.30 AM.png (104.6 KB) - added by kraftbj 7 years ago.
25426.patch (683 bytes) - added by kraftbj 7 years ago.
Reducing margin-bottom, pass 1
25426.patch-inaction.png (24.2 KB) - added by kraftbj 7 years ago.
25426-1.patch (367 bytes) - added by kraftbj 7 years ago.
Refreshed patch. Only removed margin-bottom: 36px per trac. Other values same as trunk.

Download all attachments as: .zip

Change History (8)

#1 follow-up: @kraftbj
7 years ago

I've knocked it down to 24px for .widgets and #secondary .widgets, as well as removed the extra margin-bottom: 36px that was added to the post format links.

Attaching patch and screenshot of it in action.

@kraftbj
7 years ago

Reducing margin-bottom, pass 1

#2 in reply to: ↑ 1 @iamtakashi
7 years ago

Replying to kraftbj:

I've knocked it down to 24px for .widgets and #secondary .widgets, as well as removed the extra margin-bottom: 36px that was added to the post format links.

The extra margin-bottom: 36px; needs to be removed but let's keep the 48px bottom margins for each widgets.

@kraftbj
7 years ago

Refreshed patch. Only removed margin-bottom: 36px per trac. Other values same as trunk.

#3 @lancewillett
7 years ago

  • Keywords has-patch added
  • Milestone changed from Awaiting Review to 3.8

#4 @lancewillett
7 years ago

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

In 25729:

Twenty Fourteen: reduce post widget margin, props kraftbj, closes #25426.

Note: See TracTickets for help on using tickets.