Closed joshdarby closed 4 years ago
Thanks, @joshdarby! let's reduce the spacing between the byline and title by roughly 1/3 and also make the categories smaller: smaller font (let's do same as byline size) and shave off the padding on the top and bottom a bit so the overall orange part is less tall/takes up less space. we want the first thing your eye to see to be the Title, not the button. (we may want to change to a different color from the orange as well, I'll ask @MirandaEcho about this.)
@joshdarby also, for the footer version:
Actually, let's remove "learn more" and just make the logo clickable to wherever "learn more" was going to go... (if that's possible) @joshdarby
@kaylima https://github.com/INN/umbrella-inndev/pull/135/commits/5ec09c2336a0ba93c92a76ea0d08e36e3102e1af updates the embed with these changes:
Homepage and footer:
Sidebar:
Changes
This pull request makes the following changes:
In content:![Screen Shot 2019-12-19 at 12 47 03 PM](https://user-images.githubusercontent.com/18353636/71202392-186e9700-226a-11ea-8015-7854a4d76708.png)
Footer:![Screen Shot 2019-12-19 at 12 46 03 PM](https://user-images.githubusercontent.com/18353636/71202393-186e9700-226a-11ea-816b-948a7d64775c.png)
Sidebar:![Screen Shot 2019-12-19 at 12 48 36 PM](https://user-images.githubusercontent.com/18353636/71202391-186e9700-226a-11ea-8687-ffe9e6f00e9e.png)
Why
For #127
Testing/Questions
Features that this PR affects:
Questions that need to be answered before merging: