-
Notifications
You must be signed in to change notification settings - Fork 38
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鈥檒l occasionally send you account related emails.
Already on GitHub? Sign in to your account
Evaluate SVG animation to replace the throbber and use consistently in core #6524
Comments
@klonos already collected the usual Drupal links for that, in case someone's interested:
Slightly related, but actually about a JS detail: #6441 |
One of the Drupal links differentiates between the throbber and loader icon. Is that true? Do we need to replace both? |
This might be a good starting point https://gist.github.com/joelpittet/c75ff7c2f86282ff9499 |
AFAIK, we're in no way bound to Drupal decisions. 馃槣 Note: as this is almost exclusively a design topic, I won't provide a PR. For sure not, if there's nothing decided, yet. |
Relevant to a11y:
Indicate the loading state of a component or page with Bootstrap spinners, built entirely with HTML, CSS, and no JavaScript: https://getbootstrap.com/docs/4.2/components/spinners |
Description of the bug
We have SVG icons in core! 馃帀
But wait, there's this ugly little throbber, inherited from old Drupal, last time adapted back in 2015, but still in "Drupal blue" and sort of 90-s style.
https://github.com/backdrop/backdrop/blob/1.x/core/misc/throbber-active.gif
And there's yet another throbber - the one that views uses when opening views field dialogs. Different colors, different style.
https://github.com/backdrop/backdrop/blob/1.x/core/misc/loading-small.gif
Now that we dropped IE support, and browser support for animated SVG is pretty good, we could consider switching to animated SVG.
Advantage: better scaling, true transparency support, so it can look good on any background-color, in any size.
Disadvantage: to be determined
First step: define requirements
The text was updated successfully, but these errors were encountered: