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

Grammar/typo fixes in Readme #7584

Merged
merged 1 commit into from
Jul 28, 2019
Merged

Conversation

Jimmydalecleveland
Copy link
Contributor

Issue: Quite a few grammatical errors and typos that made the readme harder to parse

What I did

Fixed all the issues I saw

@vercel
Copy link

vercel bot commented Jul 27, 2019

This pull request is automatically deployed with Now.
To access deployments, click Details below or on the icon next to each push.

Latest deployment for this branch: https://monorepo-git-fork-jimmydalecleveland-patch-2.storybook.now.sh

addons). If the active param were gone after story switching, it fallback to the default then the first. As a
rule of thumbs, whenever collisions made possible, always the first wins.
5. Query parameters are supported for pre-selecting contexts param, which comes handy for visual regression testing.
addons). If the active params were gone after story switching, it falls back to the default then the first. As a
Copy link
Member

Choose a reason for hiding this comment

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

nit: fallbacks

@@ -178,13 +178,13 @@ An array of params contains a set of predefined `props` for `components`. This o

(required)

An unique name for representing the props.
A unique name for representing the props.
Copy link
Member

Choose a reason for hiding this comment

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

not a grammar pro but shouldnt this stay An as unique starts with a vowel?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

It is a vowel, but the sound is what matters. A consonant sound, which unique makes here ("yun" with a hard y).

Copy link
Member

@shilman shilman left a comment

Choose a reason for hiding this comment

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

Great updates. Thanks!! 🙏

Copy link
Member

@shilman shilman left a comment

Choose a reason for hiding this comment

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

Great updates! Thanks!! 🙏

@shilman shilman merged commit 78eaefa into storybookjs:next Jul 28, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants