Skip to content

Conversation

@simonbyford
Copy link
Contributor

@simonbyford simonbyford commented Jan 22, 2026

What does this change?

Second attempt at not showing cursor: pointer when hovering over Cinemagraphs on articles.

Why?

My first attempt was overcomplicating things by distinguishing between Cinemagraphs on articles vs fronts.

Cinemagraphs on fronts have their cursor: pointer applied further up the tree so this change should only affect Cinemagraphs on articles.

Screenshots

hover

@github-actions
Copy link

github-actions bot commented Jan 22, 2026

@domlander
Copy link
Contributor

domlander commented Jan 22, 2026

Looks good! When cinemagraphs are used in fronts, the cursor: pointer comes from the card link, so this change won't affect either fronts cards or feature cards. I've tested this locally.

@simonbyford simonbyford marked this pull request as ready for review January 22, 2026 17:11
@github-actions
Copy link

Hello 👋! When you're ready to run Chromatic, please apply the run_chromatic label to this PR.

You will need to reapply the label each time you want to run Chromatic.

Click here to see the Chromatic project.

@simonbyford simonbyford added this to the Requests milestone Jan 22, 2026
@simonbyford simonbyford self-assigned this Jan 22, 2026
@simonbyford simonbyford added the run_chromatic Runs chromatic when label is applied label Jan 23, 2026
@github-actions github-actions bot removed the run_chromatic Runs chromatic when label is applied label Jan 23, 2026
@simonbyford simonbyford force-pushed the cinemagraph-cursor-fix branch from 5637858 to e90ae5b Compare January 23, 2026 10:17
@simonbyford simonbyford added the run_chromatic Runs chromatic when label is applied label Jan 23, 2026
@github-actions github-actions bot removed the run_chromatic Runs chromatic when label is applied label Jan 23, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

fix Departmental tracking: fix

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants