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

CRM-21241 Logo improvements #11046

Merged
merged 3 commits into from
Oct 2, 2017
Merged

CRM-21241 Logo improvements #11046

merged 3 commits into from
Oct 2, 2017

Conversation

agh1
Copy link
Contributor

@agh1 agh1 commented Sep 30, 2017

Overview

The spinning CiviCRM logo in the top menu is bumpy and annoying. Also, the spinning logo as modals load is too fast and jittery.

This also removes a file used only in WordPress sites. civicrm/civicrm-wordpress#118 should be merged first--it removes the references to it.

Before

  • The logo in the top menu jumps around instead of spinning smoothly
  • The center of the top menu logo's triangles is not the center of the spinning sprite. The result is that the logo appears to flop around.
  • The spinning logo that appears in modals as they load is monochromatic and spins rapidly.

After

  • The logos spin smoothly at approximately half a revolution per second.

Technical Details

The menu logo was given both fa-spin and fa-pulse classes. The latter makes it rotate in steps rather than turning smoothly. The new GIF for modals now has more steps and moves more slowly among them.


----------------------------------------
* CRM-21241: Spinning logo is too jumpy
  https://issues.civicrm.org/jira/browse/CRM-21241
----------------------------------------
* CRM-21243: Logo in WP menu doesn't follow standard
  https://issues.civicrm.org/jira/browse/CRM-21243y
…color

----------------------------------------
* CRM-21241: Spinning logo is too jumpy
  https://issues.civicrm.org/jira/browse/CRM-21241
@agh1
Copy link
Contributor Author

agh1 commented Sep 30, 2017

@tommybobo this is a favor for you

@colemanw colemanw merged commit dce0aa6 into civicrm:master Oct 2, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants