Skip to content

Delete from Grid step layout - #4942

Open
junyuanxue wants to merge 1 commit into
09-14-delete_from_content_stepfrom
09-15-delete_from_grid_step
Open

junyuanxue wants to merge 1 commit into
09-14-delete_from_content_stepfrom
09-15-delete_from_grid_step

Conversation

@junyuanxue

@junyuanxue junyuanxue commented Sep 15, 2026

Copy link
Copy Markdown
Contributor

What does this change?

  • Add 'delete from grid' step, UI only and not yet wired up with delete API calls (to be handled upstack), to keep the code review easier
  • This step will render a list of crops and usages, allow user to select from 1 out of 3 delete methods, then confirm the deletion
  • I've opted for a simple, flat list of usages display, with downloads collapsed into 'x downloads'. This is not consistent with 'Usages' tab but as this just serves as a quick confirmation before the user proceeds to nuke the image, I think we should stick to whatever is the simplest
  • I needed to add a security.styleSources conf so we can load Material Symbols from @guardian/stand, see icon doc
  • I have restructured the theme overrides a little so we clearer type definitions

How can success be measured?

  • It's a bit annoying to test locally due to /capiUsages endpoint calling Preview CAPI, so best way is to test with a deployment in TEST
  • Go to an image page, and append '/takedown' at the end of the url. You'll need the 'delete_crops_or_usages' permission to access it
delete from grid

Tested? Documented?

  • locally by committer
  • locally by Guardian reviewer
  • on the Guardian's TEST environment
  • relevant documentation added or amended (if needed)

Stack created with GitHub Stacks CLIGive Feedback 💬


@junyuanxue
junyuanxue added this pull request to stack #4936 September 15, 2026 16:35
@junyuanxue junyuanxue added the feature Departmental tracking: work on a new feature label Sep 15, 2026
@github-actions

github-actions Bot commented Sep 15, 2026

Copy link
Copy Markdown

@junyuanxue
junyuanxue force-pushed the 09-15-delete_from_grid_step branch 2 times, most recently from 0be0b1a to c3310c6 Compare September 16, 2026 13:34
@junyuanxue
junyuanxue force-pushed the 09-15-delete_from_grid_step branch 2 times, most recently from 56a55a6 to 693df3a Compare September 17, 2026 16:04
@junyuanxue
junyuanxue force-pushed the 09-15-delete_from_grid_step branch from 693df3a to 20f14e5 Compare September 18, 2026 11:25
@junyuanxue junyuanxue changed the title Delete from Grid step Delete from Grid step layout Sep 18, 2026
@junyuanxue
junyuanxue marked this pull request as ready for review September 18, 2026 15:15
@junyuanxue
junyuanxue requested a review from a team as a code owner September 18, 2026 15:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feature Departmental tracking: work on a new feature

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant