Bug #11506

Home Page Slideshow - blank slide

Added by scott voth 4 months ago. Updated 3 months ago.

Priority name:
Category name:
Homepage Slides
Target version:
Start date:
Due date:
% Done:


Estimated time:


Hero slideshow seems to have a blank slide at the beginning:

<img alt='css.php' src="" width='1' height='1' /> <div class="slide first_slide">

Related issues

Related to CUNY Academic Commons - Bug #11453: 403 error on FirefoxResolved2019-05-14


#1 Updated by Boone Gorges 4 months ago

  • Status changed from New to Assigned
  • Assignee changed from scott voth to Raymond Hoh
  • Target version set to 1.15.3

Thanks, Scott. This stems from Ray's fix in The Cookies for Comments image was being triggered for the very first WP_Query loop on the page, which in the case of the homepage appears to be the hero slider. I made a quick change in that limits the loop_start and loop_no_results callbacks so that they only add the image if we're in the main query. I believe this will avoid problems like this more generally, but it definitely needs review from Ray :)

#2 Updated by Boone Gorges 4 months ago

  • Related to Bug #11453: 403 error on Firefox added

#3 Updated by Raymond Hoh 4 months ago

Good catch, Boone.

The main query limitation was something I considered, but thought it would be better to open up the restriction so the Cookies for Comments asset had a better chance of being rendered before the footer.

Currently, if you check the Cookies for Comments asset on the site (the site referenced in #11453), the asset is now loaded in the footer instead of before.

Maybe, instead, I can check the post type and bail if the post type matches 'slide'?

#4 Updated by Boone Gorges 4 months ago

I'd considered the 'slide' blacklist, but it seems like this leaves us open to whack-a-mole for other similar problems.

That being said, this is a very specific problem to the homepage slider plugin, which parses body content for img tags. This is very unusual, and is unlikely to arise in many other cases. So maybe the blacklist is OK.

Happy to go with whatever you think best.

#5 Updated by Boone Gorges 3 months ago

  • Target version changed from 1.15.3 to 1.15.4

#6 Updated by Boone Gorges 3 months ago

  • Target version changed from 1.15.4 to 1.15.5

#7 Updated by Raymond Hoh 3 months ago

  • Status changed from Assigned to Resolved

Let's mark this as resolved for now.

Also available in: Atom PDF