I've tested out the color palette feature and it's great, but there didn't seem much point in keeping it tucked away inside the configuration menu. So I have created a patch that adds it to the main style guide page.

CommentFileSizeAuthor
#3 styleguide-1928640-3.patch2.46 KBtim.plunkett
#3 interdiff.txt2.59 KBtim.plunkett
#1 add_color_palette_to_style_guide_page-1928640-2.patch928 bytesjptaranto
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

jptaranto’s picture

jptaranto CreditAttribution: jptaranto commented
Status: Active » Needs review
FileSize
add_color_palette_to_style_guide_page-1928640-2.patch928 bytes
rupl’s picture

rupl
he/him
Primary language English
CreditAttribution: rupl commented
Status: Needs review » Reviewed & tested by the community

This patch works exactly as described. RT to the BC!

tim.plunkett’s picture

tim.plunkett
he/him
Primary language English
Location Philadelphia
CreditAttribution: tim.plunkett commented
FileSize
interdiff.txt2.59 KB
styleguide-1928640-3.patch2.46 KB

I think it's probably worth a function to properly loop through and sanitize the swatches (and cast to an array). Leaving RTBC.

dead_arm’s picture

dead_arm
she/her
Primary language English
CreditAttribution: dead_arm commented
Status: Reviewed & tested by the community » Fixed

Thanks everyone! Committed! http://drupalcode.org/project/styleguide.git/commit/e51df2e3bd23da86365d...

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.