Automattic / Automattic/jetpack
Port Carousel to work with AMP
Open
[Feature] Carousel
[Pri] Low
AMP
Enhancement
- Dominant language
- PHP
- Stars
- 1.8k
- Forks
- 898
- Avg merge
- 1d 18h
- Merged PRs (30d)
- 774
Description
In #9458 we propose disabling Carousel as it's broken on AMP sites, primarily due to the reliance on javascript.
This issue was created to track making this feature work with AMP.
AMP has a built-in Carousel, so this shouldn't be too hard.
Contributor guide
Research direction
No source file or test is named. Start by reading issue #9458 and the AMP Carousel documentation, then locate the existing Carousel entry point and its AMP integration constraints; done means Carousel works on AMP sites without relying on unsupported JavaScript.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- javascript
- Domain
- frontend
- Issue type
- Feature
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Needs clarification
- Newbie friendliness
- 25/100