Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Convert Community page to Tachyons and make it Responsive. #362

Merged
merged 9 commits into from
Nov 27, 2018

Conversation

chriskrycho
Copy link
Contributor

@chriskrycho chriskrycho commented Nov 20, 2018

Notes:

  • This supersedes fix(accessibility): render images with img not backgrounds. #360, as I ended up just starting this on top of that branch because I needed that as the starting point for working on this page (and didn't want to lose the work I'd already done there).

  • This includes replacing container with equivalent Tachyons CSS everywhere. We likely want to revisit how that style is applied, as it's otherwise going to be a copy-paste thing everywhere. However, it's now one less piece using Skeleton, and it's also actually quite useful to build this way, given how it lets us do things like I did here and enlarging the first major section's text to have better typographical rhythm.

small phone

small-phone

tablet (landscape)

tablet-landscape

tablet (portrait)

tablet-portrait

laptop

laptop

widescreen

widescreen

- Decrease the specificity of the list styles in `app.scss`.
- Update the community page list of RustBridge images to use a list
  instead of a bunch of nested `div`s.
Use an equivalent Tachyons declaration to the base `container`
declaration from Skeleton. This is var more verbose, but allows
setting individual properties more specifically later (rather than
having to use overrides).
@chriskrycho chriskrycho added this to the beta launch milestone Nov 20, 2018
@ashleygwilliams ashleygwilliams merged commit 2d766a6 into master Nov 27, 2018
@delete-merged-branch delete-merged-branch bot deleted the responsive-tachyons-community-233 branch November 27, 2018 20:16
This was referenced Nov 28, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants