Skip to content

chore: setting up project (cont) - #2

Open
cameronolivier wants to merge 6 commits into
masterfrom
getting-to-build
Open

chore: setting up project (cont)#2
cameronolivier wants to merge 6 commits into
masterfrom
getting-to-build

Conversation

@cameronolivier

Copy link
Copy Markdown
Contributor

Work Done

chore: adding React and React-Native as devDeps and peerDeps
chore: fixing up storybook components
chore: removed DevInfoPane component for now (doesn't add much utility atm)
refactor: moved the Storybook Constants into its own file.
refactor: moved all colors into individual exports to easier see which color is being used.
refactor: moved all Storybook components into individual folders with index.js for exports.
feat: brought in utils/propTypes.utils.js.


export const lightBlue = '#23A5DD'
export const blue = '#007AFF'
export const blueDark = '#2269A3'

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Is this going to be the same for every project?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

these can be removed - was keeping in the off-chance we needed it for within the components - but I think best to just remove altogether - only need black and dark grey I think
I can do a cleanup now or in a new PR?

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I think could you remove what you don't want here and push a commit?

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