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

Several text tweaks in README #30

Closed
wants to merge 5 commits into from

Conversation

ppowalowski
Copy link
Contributor

Cleaned some typos and tweaked phrasing.

There is an event that is fired on the modal itself after the modal is show
which is called `cssmodal:show` and an event after the modal is hidden called
`cssmodal:hide`.
There is an event `cssmodal:show` fired on the modal itself after
Copy link
Contributor

Choose a reason for hiding this comment

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

I like this description over the initial.

@anselmh
Copy link
Contributor

anselmh commented May 12, 2013

I personally prefer the current (not PR) text except for the one sentence I commented on in diff. But this is just my opinion.

@drublic
Copy link
Owner

drublic commented May 12, 2013

Thanks for the contribution. I've tried my best and cherry picked cross commits. It's modified in b959efc. Wheee!

@drublic drublic closed this May 12, 2013
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.

None yet

3 participants