thinktandem / tandem

All the things that power the Tandem engine
https://thinktandem.io/
3 stars 2 forks source link

Thiago Website Fixes: Components Section #232

Open labboy0276 opened 3 years ago

labboy0276 commented 3 years ago

Site Header — #header

Final copy should look like this.

Mobile Breakpoints

Homepage Section — .fp-section

Hero CTA — .hero-type

Homepage Intro — .fp-section:first-oftype

This is what is should look like.

Page Title — .section-header-left h1

font-size: 32px;
font-weight: 900;
border-bottom: 0.5rem solid #000;
display: inline-block;
padding-bottom: 0.5em;

Section Header — .section-header

Big title, for people with big monitors, and imacs, lets go bold with some 96px titles (we can play with how dramatic we make it, but do look at the screenshot linked above. With a little tweaking, we can maybe have that work everywhere, or come up with a type scale that works. Last bit of emphasized text, feels like it should maybe link to something, and that cue is reinforced on the homepage. Maybe just vertical space between it and the previous paragraph, but keeping the font-weight constant, may help alleviate this? | See Screenshot » |

Custom Block Title — .custom-block-title

Work Grid — .work-grid

Same recommendations re: heading leading / letter spacing as everywhere else, beware a11y color combos and hover states.

margin-bottom: 1em;
(on logo) padding-right: 2em // instead of one but better logos from clients

Footer — #footer

Copyright line, use good ol sans-serif at smaller than 1rem? Right now it looks "designed" because Galaxie Copernicus is kinda nice, but then writing TANDEM in it, is kinda off-brand. Should be aligned to the baseline of the secondary nav, rather than vertically centered in this space. Remove bracket hover state (keep mikes unique creation)

labboy0276 commented 3 years ago

I did the best I could with these requests, I will wait for Thiago feedback. a11y stuff can wait

PR: https://github.com/thinktandem/tandem/pull/238