Skip to content
This repository has been archived by the owner on Feb 18, 2022. It is now read-only.

Add global styles + base for form page #2

Merged
merged 3 commits into from Mar 28, 2020
Merged

Add global styles + base for form page #2

merged 3 commits into from Mar 28, 2020

Conversation

notfelineit
Copy link
Contributor

@notfelineit notfelineit commented Mar 28, 2020

Add Global Styles & Form

This PR:

  • adds global styles based on Juan's css file
  • adds a base for the form page
  • adds style examples to the /test page for each css style

Styles

The styles are the following:
Screen Shot 2020-03-27 at 9 48 22 PM

We can update them as we build out the real pages! 馃槃

Copy link
Contributor

@szhu szhu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

馃敟

src/styles.ts Outdated
@@ -0,0 +1,101 @@
import { createGlobalStyle } from "styled-components";
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

can we name all our files .tsx? This avoids situations where we rename a file .ts -> .tsx after we realize we need to put jsx in it.

also, the linters that are set up are currently only checking .tsx files. alternatively we can just add .ts patterns.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sure thing!

@notfelineit notfelineit merged commit b8c1b6c into master Mar 28, 2020
@notfelineit notfelineit deleted the franny/form branch March 28, 2020 04:56
@szhu szhu mentioned this pull request Mar 30, 2020
Comment on lines +4 to +6
@import url('https://fonts.googleapis.com/css?family=Poppins&display=swap');
@import url('https://fonts.googleapis.com/css?family=Libre+Baskerville&display=swap');
@import url('https://fonts.googleapis.com/css?family=Rubik&display=swap');
Copy link
Contributor

@szhu szhu Mar 30, 2020

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Creating #10 to remove this because we realized it didn't work, and because styled-components gives a warning about not doing this.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants