Automattic / Automattic/jetpack

Tiled Gallery block: Hide 'Link To' options if Carousel is enabled

Open
#42,097 0 comments 0 reactions 0 assignees View on GitHub
[Block] Tiled Gallery [Feature] Carousel [Feature] Tiled Gallery [Pri] Normal Enhancement
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

Open the contributing 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

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.