Automattic / Automattic/jetpack
Tiled Gallery block: Hide 'Link To' options if Carousel is enabled
- Dominant language
- PHP
- Stars
- 1.8k
- Forks
- 898
- Avg merge
- 1d 18h
- Merged PRs (30d)
- 774
Description
### Suggestion
If Carousel is enabled, then by default none of the 'Link To' options for Tiled Gallery block images will work, the Carousel taking precedence.
It may make more sense to hide these 'Link To' options when Carousel is enabled, perhaps adding a link to enable / disable Carousel, along with an explanation of why these options aren't showing.
However for custom links in particular (newly added here - https://github.com/Automattic/jetpack/pull/41471), it may make more sense for the images within Carousel's to include the custom links as well (see issue https://github.com/Automattic/jetpack/issues/42096), in which case showing the custom link option would still make sense.
Mentioned in internal testing: p8oabR-1Di-p2#comment-8406
Contributor guide
Research direction
Start with the Tiled Gallery block's Carousel and “Link To” controls, then review PR #41471 and issue #42096 for the custom-link context. The issue needs a settled choice between hiding the options or carrying custom links into the Carousel; done means the selected behavior is implemented and explained consistently.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- javascript, react, wordpress
- Domain
- frontend, web-dev
- Issue type
- Feature
- Difficulty
- 5/5
- Estimated time
- Over a week
- Activity status
- Stale
- Clarity
- Needs clarification
- Newbie friendliness
- 25/100