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

Add pulsing spin animation #148

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from
Open

Conversation

1zaman
Copy link
Contributor

@1zaman 1zaman commented Dec 16, 2015

The Font Awesome CSS framework has a pulse animation that rotates an icon in 8 steps, which matches the structure of it's pulsing spinner. This has been implemented with the "pulse" token here. Since Font Awesome seems to have the only dotted pulsing spinner out of the current font set, this seems a reasonable generic implementation at the moment. If at some point spinners with different pulses need to be handled, then a pulse property can be introduced for the icons, with 8 as the default count.

The Font Awesome CSS framework has a pulse animation that rotates an
icon in 8 steps, which matches the structure of it's pulsing spinner.
This has been implemented with the "pulse" token here. Since Font
Awesome seems to have the only dotted pulsing spinner out of the current
font set, this seems a reasonable generic implementation at the moment.
If at some point spinners with different pulses need to be handled, then
a pulse property can be introduced for the icons, with 8 as the default
count.
@1zaman
Copy link
Contributor Author

1zaman commented Dec 16, 2015

Note that I have a few pull requests open at the moment, and they all somewhat affect each other. My develop branch can be consulted (and used) in merging multiple of these pull requests, as I have merged all of them these with appropriate summary in the merge commit messages. Alternatively, if you merge them one at a time, then I'll rebase the others to the merged state.

@andreasschiestl
Copy link

please merge this feature. It helps to make Apps that are consistent between Web and Android.

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