-
Notifications
You must be signed in to change notification settings - Fork 799
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Slideshow Block: AMP Compatibility (#13009)
* Slideshow block, AMP compatibility, initial commit. * Background image data URIs don't seem to work in AMP unless spaces are encoded. * Aria labels for next and previous buttons. * Slideshow pagination control in AMP. * Safer global for Slideshow ID. * Attribute and caption escaping. * Extract Slideshow AMP rendering to a separate function. * Use wp_get_attachment_image() and let image sanitizer manage. Use 'large' size rather than 'full.' * Additional attribute escaping. * Bug fix: remove autoplay override used for testing. * Unnecessary style removal. * Autoplay pause/play button implementation for AMP. * Set AMP Carousel dimensions based on the aspect ratio of the first image. * Use static variable instead of Global for Slideshow block unique ID. * Some refactoring to reduce size of individual functions. * Further refactoring to reduce function length. * Remove negative top margin for next/prev buttons in AMP requests. * Selected state for Slideshow block bullets in AMP requests. * Remove short array syntax. * Bug fix: incorrect element ID used in autoplay pause/play buttons. * Improvements to formatted string statements. * Escaping improvements for Slideshow block bullets. * Check for existence of image dimensions, use 800x600 as fallback. * Use Jetpack_Gutenberg's block_classes to generate block class list. * Improved approach to translating bullet ARIA labels.
- Loading branch information
Jefferson Rabb
authored
Oct 3, 2019
1 parent
7671f73
commit 4a04ac2
Showing
2 changed files
with
205 additions
and
7 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters