Opened 2 months ago

Closed 2 months ago

Last modified 6 weeks ago

#23784 closed enhancement (fixed)

Twenty Thirteen: Post format and link color tweaks for consistency

Reported by: Joen Owned by: lancewillett
Priority: normal Milestone: 3.6
Component: Bundled Theme Version: trunk
Severity: normal Keywords: has-patch
Cc:

Description

I discovered that a couple of the post format background colors (video, audio, gallery, quote) were slightly off compared to mockups: http://cl.ly/image/3b0O21090o2E — it's a small change, probably due to a color profile mismatch on my part. The new colors are a *teensy* bit more muted.

Additionally, link colors should be consistent. For example, standard, link and aside post formats should have same link color for meta and entry-content, for the sake of simplicity and consistency.

The updated colors in this patch pass WebAIM color contrast checker (http://webaim.org/resources/contrastchecker/) to ensure proper readability.

Attachments (1)

colortweaks.diff (3.1 KB) - added by Joen 2 months ago.
Patch to tweak off post format colors, and make link colors consistent.

Download all attachments as: .zip

Change History (9)

Joen2 months ago

Patch to tweak off post format colors, and make link colors consistent.

  • Milestone changed from Awaiting Review to 3.6

comment:2 follow-up: ↓ 5   lancewillett2 months ago

Joen: changes look good, thank you.

I assumed this selector change was not intentional, though:

.format-quote .entry-content .page-links a, 

so I left it out.

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

In 23756:

Twenty Thirteen: post format and link color tweaks for consistency, props joen. Fixes #23784.

Two small notes for making patches for core Trac:

  1. Make the patch from WordPress root (not inside the theme directory)
  2. Name it [the ticket number].diff

Both those help patches like me a ton applying and testing out lots of patches. :)

comment:5 in reply to: ↑ 2   Joen2 months ago

Replying to lancewillett:

Joen: changes look good, thank you.

I assumed this selector change was not intentional, though:

.format-quote .entry-content .page-links a, 

so I left it out.

I think it was actually intentional — otherwise I get this on my test setup:
http://cl.ly/image/3u261v3N3I2I

Thanks for the info on the patches, this is actually my first actual core contribution. I'm making a mental note, because I hope to make many more contributions.

  • Resolution fixed deleted
  • Status changed from closed to reopened
  • Resolution set to fixed
  • Status changed from reopened to closed

In 23771:

Twenty Thirteen: quote post format color fix for paging links, props Joen. Closes #23784.

In 23962:

Twenty Thirteen: minor design tweaks and fixes:

  • Link color should be the same on all links that are on white or tan backgrounds, see #23784
  • Better color value for active menu item (red instead of tan)
  • Improved styles for quote citations that are wrapped in links
  • Better layout for single post navigation links when post titles are really long
  • Fix navbar border style for viewports at 1600 pixels wide and above

Props Joen and obenland.

Note: See TracTickets for help on using tickets.