Make WordPress Core

Opened 7 years ago

Closed 7 years ago

#49546 closed defect (bug) (fixed)

Twenty Thirteen: Update calendar widget styles for 5.4 markup

Reported by: sabernhardt Owned by: ianbelanger
Priority: normal Milestone: 5.4
Component: Bundled Theme Version:
Severity: normal Keywords: has-patch has-screenshots
Cc: Focuses: css

Description

Since the previous month and next month links are no longer inside table cells in version 5.4, display:block stretches each of those links the full width of the calendar container.

Though I like how the month links are larger and further away from the calendar date links, the change would be quite unexpected for other people.

(markup was changed for ticket:39763)

Attachments (6)

2013-side-comparison.jpg​ (74.6 KB ) - added by sabernhardt 7 years ago.
comparing side widget in 5.3 and 5.4
2013-footer-comparison.jpg​ (67.1 KB ) - added by sabernhardt 7 years ago.
comparing footer widget in 5.3 and 5.4 (at narrow width so they are aligned in one column instead of using masonry)
49546.patch​ (599 bytes ) - added by sabernhardt 7 years ago.
Using display:table to nearly match the former styles
49546.1.patch​ (433 bytes ) - added by sabernhardt 7 years ago.
Using display:inline-block for the new span elements
2013-side-patches.jpg​ (117.8 KB ) - added by sabernhardt 7 years ago.
comparing patches for side widget next to 5.3 styles
2013-footer-patches.jpg​ (108.6 KB ) - added by sabernhardt 7 years ago.
comparing patches for footer widget (again, in one column) next to 5.3 styles

Download all attachments as: .zip

Change History (14)

@sabernhardt
7 years ago

comparing side widget in 5.3 and 5.4

@sabernhardt
7 years ago

comparing footer widget in 5.3 and 5.4 (at narrow width so they are aligned in one column instead of using masonry)

@sabernhardt
7 years ago

Using display:table to nearly match the former styles

@sabernhardt
7 years ago

Using display:inline-block for the new span elements

#1 @SergeyBiryukov
7 years ago

  • Milestone Awaiting Review → 5.4

@sabernhardt
7 years ago

comparing patches for side widget next to 5.3 styles

@sabernhardt
7 years ago

comparing patches for footer widget (again, in one column) next to 5.3 styles

#2 @sabernhardt
7 years ago

Option 1: 49546.patch

  • Uses display:table and display:table-cell to nearly match the former styles (the .pad in between the links is slightly narrower)
  • Reinstates the line-height of 2 for the navigation links
  • Moves the table's bottom margin to the nav element

Option 2: 49546.1.patch

  • Simply adds display:inline-block to the new span elements (it may be preferable to add that to the .wp-calendar-nav span a elements instead)

This ticket was mentioned in ​Slack in #core-themes by sabernhardt. ​View the logs.


7 years ago

#4 @ianbelanger
7 years ago

  • Keywords commit added
  • Owner set to ianbelanger
  • Status new → reviewing
  • Version trunk

Reviewing for commit.

#5 @ianbelanger
7 years ago

Patch looks good @sabernhardt nice find! I'm going to commit your first patch as it more aligns with how the calendar looked before 5.4.

#6 @ianbelanger
7 years ago

  • Resolution → fixed
  • Status reviewing → closed

In 47392:

Bundled Themes: Twenty Thirteen update calendar widget styles for 5.4 markup.

Fixes the alignment of month links in the calendar widget with the changes in version 5.4

Props sabernhardt.
Fixes #49546.

#7 @ianbelanger
7 years ago

  • Keywords fixed-major added; commit removed
  • Milestone 5.4 → 5.3.3
  • Resolution fixed
  • Status closed → reopened

Reopening for backport.

#8 @ianbelanger
7 years ago

  • Keywords fixed-major removed
  • Milestone 5.3.3 → 5.4
  • Resolution → fixed
  • Status reopened → closed

Doesn't need backport since the issue appeared in 5.4.

Note: See TracTickets for help on using tickets.