Edit youtube js file structure in heroslider to make it work better
and avoid some issues appear when the page takes time to load like the video not autoplay in the first slider swipe.

CommentFileSizeAuthor
#6 Test Varbase hero slider YouTube Video Slides - after the better refactor change of logic.gif16.85 MBRajab Natshah
#3 3088950-edit-youtube-js-structure.patch4.8 KBfirass.ziedan
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Firass Ziedan created an issue. See original summary.

firass.ziedan’s picture

firass.ziedan
Location Jordan
CreditAttribution: firass.ziedan at Vardot commented
Status: Active » Needs work
firass.ziedan’s picture

firass.ziedan
Location Jordan
CreditAttribution: firass.ziedan at Vardot commented
Status: Needs work » Needs review
FileSize
3088950-edit-youtube-js-structure.patch4.8 KB
Rajab Natshah’s picture

Rajab Natshah CreditAttribution: Rajab Natshah at Vardot for Vardot commented
Title: Edit youtube js file structure » Change Youtube js file structure in hero slider to make it work better and avoid some issues

Rajab Natshah’s picture

Rajab Natshah CreditAttribution: Rajab Natshah at Vardot for Vardot commented
Assigned: Unassigned » Mohammed J. Razem
Issue tags: +varbase-8.7.3
FileSize
Test Varbase hero slider YouTube Video Slides - after the better refactor change of logic.gif16.85 MB

Committed ... Thank you Firas :)
It is a better refactor change of logic.
Fully tested the change of the behaviour

https://youtu.be/XhJgBiLbwtw

Test Varbase hero slider YouTube Video Slides after the better refactor change of logic

Rajab Natshah’s picture

Rajab Natshah CreditAttribution: Rajab Natshah at Vardot for Vardot commented
Assigned: Mohammed J. Razem » Unassigned
Rajab Natshah’s picture

Rajab Natshah CreditAttribution: Rajab Natshah at Vardot for Vardot commented
Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

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