I have a view of users registered in the site with their images and items per page to 50 users.
Exposed filter in the view for filtering by name and city.
I load the page and scroll down all images are loaded and if I use the exposed filters to search for users it works fine.
If i load the page and without scrolling down use the exposed filters to search for a user and then scroll down the images that did not load initially have an extra spinner near them.
Every time i change my filter without loading all the images in the page an extra spinner is added.

Image attached.

CommentFileSizeAuthor
#7 multiple_spinners_added-2824613-7.patch758 byteslegolasbo
#2 lazy.png35.65 KBSantVim
#4 multiple_spinners_added-2824613-4.patch1.04 KBfinnsky
#5 interdiff.txt418 bytessergeimalyshev
#5 multiple_spinners_added-2824613-5.patch1.01 KBsergeimalyshev
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

SantVim created an issue. See original summary.

SantVim’s picture

SantVim CreditAttribution: SantVim commented
Issue summary: View changes
FileSize
lazy.png35.65 KB
SantVim’s picture

SantVim CreditAttribution: SantVim commented

Tried again with unload option checked, Every time I scroll up and down the spinner count reduces and disappears.

finnsky’s picture

finnsky CreditAttribution: finnsky at Skilld commented
FileSize
multiple_spinners_added-2824613-4.patch1.04 KB

steps to reproduce:

  1. Enable views AJAX.
  2. enable "Expose items per page".
  3. Change selectbox filter option.

Addd fix to check if icon-gif added already

sergeimalyshev’s picture

sergeimalyshev CreditAttribution: sergeimalyshev at Skilld commented
Status: Active » Needs review
FileSize
multiple_spinners_added-2824613-5.patch1.01 KB
interdiff.txt418 bytes

There are very many console logs if use patch #4. I think it is unnecessary.

DamienMcKenna’s picture

DamienMcKenna
Location NH, USA
CreditAttribution: DamienMcKenna at Mediacurrent commented
Status: Needs review » Reviewed & tested by the community

This appears to work nicely.

legolasbo’s picture

legolasbo
he/him
Primary language Dutch
Location Middelburg
CreditAttribution: legolasbo at Ordina Digital Services for Sijthoff Media commented
Status: Reviewed & tested by the community » Needs review
FileSize
multiple_spinners_added-2824613-7.patch758 bytes

I've just merged #2935566: Icon not using real relative positioning which causes this patch to need a reroll. While looking at the patch I've found a cleaner way to implement it, so I've rewritten it. Please verify that it still works as expected.