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

feat(readme): update readMe, update replace method #23

Merged
merged 4 commits into from Nov 12, 2018

Conversation

denStrigo
Copy link
Collaborator

No description provided.

…tions, update replace method, set icon animation as more priority than common replace.
README.md Outdated
<i data-eva="github" data-eva-animation="zoom"></i>
```

- By default animation works by hover and has infinity `false`, if you want change this behavior add `data-eva-hover="true | false"` and `data-eva-infinite="true | false"`
Copy link
Contributor

Choose a reason for hiding this comment

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

this sentence is quite hard to understand, try to rephrase

README.md Outdated
```
> **Note:** The animation will be applied to all replaced elements.

- Add class `eva-parent-hover` to parent container if you want that the animation works by hover on the parent element.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
- Add class `eva-parent-hover` to parent container if you want that the animation works by hover on the parent element.
- Add `eva-parent-hover` class to the parent container in a case you want to activate the animation hovering on the parent element.

README.md Outdated
```

- Additional animation attributes:
* `data-eva-hover`: Makes an animation available by hover. Default value is `true`. Available true or false.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
* `data-eva-hover`: Makes an animation available by hover. Default value is `true`. Available true or false.
* `data-eva-hover`: Makes the animation available on hover. Default value is `true`. Available `true` or `false`.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

done

README.md Outdated

- Additional animation attributes:
* `data-eva-hover`: Makes an animation available by hover. Default value is `true`. Available true or false.
* `data-eva-infinite`: Makes animation as infinity. Default value is `false`. Available true or false.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
* `data-eva-infinite`: Makes animation as infinity. Default value is `false`. Available true or false.
* `data-eva-infinite`: Makes the animation infinite. Default value is `false`. Available `true` or `false`.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

done

@nnixaa nnixaa merged commit cbf5a73 into akveo:master Nov 12, 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.

None yet

2 participants