y7kim / agency-jekyll-theme

Agency Theme for Jekyll
Apache License 2.0
868 stars 2.02k forks source link

Colors configuration doesnt work... #42

Closed rodrigograca31 closed 7 years ago

rodrigograca31 commented 7 years ago

Because of this commit: https://github.com/y7kim/agency-jekyll-theme/commit/730995b9e4cda4eb11564bc0ad604b1e35b42083

The "variables path" was changed from site.color.primary to site.data.template.color.primary and that was not reflected on _includes/css/bootstrap.min.css ...

I've noticed that because the form submission success message did not had a background color....

I will send a simple PR that replaces site.color.primary by site.data.template.color.primary...

rodrigograca31 commented 7 years ago

BTW, Thanks for the theme! (It's my first time using jekyll...)

y7kim commented 7 years ago

You're most welcome. Thanks for contributing!

On Tue, Jul 26, 2016 at 10:19 AM, Rodrigo Graça notifications@github.com wrote:

BTW, Thanks for the theme! (It's my first time using jekyll...)

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/y7kim/agency-jekyll-theme/issues/42#issuecomment-235339924, or mute the thread https://github.com/notifications/unsubscribe-auth/ADtkiDeYzCPGSWkg1Xto-gW1Ck_kgueTks5qZkGxgaJpZM4JVYXs .