Make loading animation play nice with Blur animation

Created on 26 September 2023, about 1 year ago
Updated 5 October 2023, about 1 year ago

Problem/Motivation

Loading animation missing/broken after updated from 2.16.0 to 2.18.0 and missing class '.is-b-loading' in the placeholder.

Steps to reproduce

Use Blazy Media field formatter for Media Image field and enabled Add is-b-visible class in the Blazy UI setting.

Proposed resolution

When page loads placeholder should have '.is-b-loading' class.

Feature request
Status

Fixed

Version

2.18

Component

Code

Created by

🇩🇪Germany ARUN AK

Live updates comments and jobs are added and updated live.
Sign in to follow issues

Comments & Activities

  • Issue created by @ARUN AK
  • 🇮🇩Indonesia gausarts

    Thank you.

    Please detail:

    • Screenshot of Blazy UI
    • Is Blur enabled?
    • Media type?
    • Media switch?

    Loading animations are indeed refined/ optimized later and disabled various conditions: iframe only, blur to avoid double animations, No JavaScript enabled, etc.

  • 🇮🇩Indonesia gausarts

    Ajaxin module will also remove is-b-loading class for its own animated elements, in case that is the case.

    However I am open to potential misses.

    Feel free to detail your expected behaviors that I missed. Thanks.

  • 🇺🇬Uganda Thomas Kaisuka

    Same Here, Animation is gone. and longer page load times than before. Its frustrating.

    I don't have Ajaxin module used anywhere

  • 🇮🇩Indonesia gausarts

    > longer page load times than before. Its frustrating.
    The loading script is almost similar to any previous / old versions, only with some betterment for quite a while since likely blazy:2.0-rc1, like using Intersection Observer and later Native lazy loading. Native browser loading managements can't be any slower.

    IMHO, loading images are determined by many factors, your internet connections, fresh or cache images, image preloading, CDN or not, DEV or PRODUCTION mode, how many concurrent connections to your server, to some extent digg/ slashdot effect, etc.

    It sometimes took me almost 1 minute to load even a page, without even images, under these conditions: drupal cache being cleared out like after running update, or even 5 minutes to infinity when drupal Update module hits its cron.

    Back to OP, I am fine to make Blur and Loading animations play nice together. The only reason the latter was removed for the first was unnecessary double animations.

    But I agreed Blur is not a loading indicator. Making this a feature instead.

    • gausarts committed 8671a649 on 8.x-2.x
      Issue #3389889 by ARUN AK, Thomas Kaisuka: Make loading animation play...
  • Status changed to Fixed about 1 year ago
  • 🇮🇩Indonesia gausarts

    I left the decision to you. The option is in Blazy UI.

    Feel free to re-open if still an issue.

    Thanks.

  • Automatically closed - issue fixed for 2 weeks with no activity.

Production build 0.71.5 2024