Skip to content
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’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[PG-621] - fix play button disappears from video #1283

Closed
wants to merge 155 commits into from
Closed

Conversation

hadasze
Copy link
Contributor

@hadasze hadasze commented Aug 27, 2024

showing play button on the video item placeholder only depends on showVideoPlayButton prop and not on the loading status of the video.

nirnaor and others added 30 commits December 15, 2022 14:58
* Revert "fix scroll indicator (#996)"

This reverts commit ca84f2b.

* Revert "refactor new sps in the gallery package (#985)"

This reverts commit edefb09.

* Revert "≈Revert "scrollDirection in gallery files""

This reverts commit a8fe658.

* Revert "scrollDirection in gallery files"

This reverts commit 9d75fb9.

* Revert "Sp playground settings (#970)"

This reverts commit 7f810ef.

* Revert "remove only in lib tests (#974)"

This reverts commit 8aef18d.

* Revert "refactor layoutHelper to new styles (#969)"

This reverts commit 6c2ed7b.

* Revert "add files for all the layouts (#968)"

This reverts commit 28ce46d.

* Revert "add new galleryLayout in presets (#967)"

This reverts commit 734617b.

* Revert "add new cosnts for all v4 defaults (#966)"

This reverts commit 983fb16.

* Revert "fix const for new scrollDirection (#965)"

This reverts commit 0eda8c6.

* Revert "add flat styles in convertion (#964)"

This reverts commit be4c11a.

* Revert "Sp styling params (#963)"

This reverts commit 85fd6d2.

* Revert "Sp behaviour params gallery horizontal (#962)"

This reverts commit 1f4f728.

* Revert "Sp behaviour params gallery vertical (#961)"

This reverts commit d967102.

* Revert "Sp behaviour params item (#960)"

This reverts commit a8948ae.

* Revert "files for layoutParams_targetItemSize (#959)"

This reverts commit a1345f8.

* Revert "add files for layoutParams_info (#958)"

This reverts commit a0c35c8.

* Revert "create files for layoutPArams_navigationArrows (#957)"

This reverts commit 9ebdc16.

* Revert "create files for layoutPArams_thumbnails (#956)"

This reverts commit c82fa6b.

* Revert "create files for layoutParams_structure (#955)"

This reverts commit a7bafd4.

* Revert "create files for layoutParams_crop (#954)"

This reverts commit 13c438f.

* Revert "add groups new sp files (#953)"

This reverts commit de818fe.

* Revert "export flat default options (#944)"

This reverts commit 5d89649.

* dummy commit

* Revert "dummy commit"

This reverts commit 299a333.

* arrows

* export
* move onclick to item container

* delete unneccery onKeyDown
GitHub Actions and others added 29 commits May 28, 2024 12:38
* fix invalid ARIA role attribute on v4

* change 'itemClick' test to match the new role change

* Store the result of this.getItemAriaRole() in a constant
* wip - ensure the correct video player is loaded

* revert change

* insert to try
* remove hls dynamic import - temporary

* add auto playVideo import

* move to componentWillMount
* get correct galleryStructure to calculate gallery width

* refresh arraow visibility - container dependent

* get the correct props on removeArrow functions
* fix highlighted thumbnail calc

* show all relevant thumbnails according to the cur activeIndex
* update deps and node version and fix jest config

* bump ts-node

* playground jsdom
@hadasze hadasze closed this Aug 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants