Update Woo Purple #227
Update Woo Purple #227
Conversation
Updating the backgrounds, logos, and favicons so we're using the right Woo Purple in the slate presentation of the docs.
jonathansadowski
left a comment
jonathansadowski
left a comment
There was a problem hiding this comment.
I just added a question about the link color, which was previously distinct from the background colors... it might be the case that it not being distinct is okay, but wanted to double-check first. I'll approve this, and if it's not an oversight, feel free to merge. Thanks for cleaning this up.
| $aside-warning-bg: #ca4949 !default; | ||
| $aside-success-bg: #38a845 !default; | ||
| $search-notice-bg: #c97a7e !default; | ||
| $link-color: #804877; |
There was a problem hiding this comment.
We were previously using a different purple here than the bg colors. Do you know if this should be the case here as well?
There was a problem hiding this comment.
@jonathansadowski good question. I noticed that it was quite similar, and I wasn't sure the reason for having two similar colours, so I used our Woo purple for both here.
old link color: 
new link color: 
old bg color: 
new bg color:
There was a problem hiding this comment.
Okay, yep, that seems reasonable to me. I'll merge this, thanks again!
Updating the backgrounds, logos, and favicons so we're using the right Woo Purple in the slate presentation of the docs.