guardian / grid

The Guardian’s image management system
https://www.theguardian.com/info/developer-blog/2015/aug/12/open-sourcing-grid-image-service
Apache License 2.0
1.44k stars 119 forks source link

make the 'graphic content blurring' explainer per-organisation #4196

Closed twrichards closed 8 months ago

twrichards commented 8 months ago

We need different organisation specific explainers for the 'graphic content' blurring, since it's 'opt-out' (see https://github.com/guardian/grid/pull/4187).

So based on the existing config property branding.staffPhotographerOrganisation, it loads the relevant HTML file from the new public/js/common/blurring directory (e.g. staffPhotographerOrganisation is GNM so loads GNM-explainer.html from that directory. This is rendered on top of the translucent overlay and the GNM implementation references the graphic-image-blur-explainer__tooltip class as before but allows for completely different styling/layout if needs be).

From a GNM perspective this is a no-op (which can be verified in TEST by deleting the SHOULD_BLUR_GRAPHIC_IMAGES cookie and reloading to reveal the explainer).

github-actions[bot] commented 8 months ago

Deploy build 12112 to TEST

All deployment options - [Deploy build 12112 to TEST](https://riffraff.gutools.co.uk/deployment/deployAgain?project=media-service%3A%3Agrid%3A%3Aall&build=12112&stage=TEST&updateStrategy=MostlyHarmless&action=deploy) - [Deploy parts of build 12112 to TEST by previewing it first](https://riffraff.gutools.co.uk/preview/yaml?project=media-service%3A%3Agrid%3A%3Aall&build=12112&stage=TEST&updateStrategy=MostlyHarmless)

From guardian/actions-riff-raff.

prout-bot commented 8 months ago

Seen on auth, usage, image-loader, thrall, cropper, collections, media-api (merged by @twrichards 8 minutes and 50 seconds ago) Please check your changes!

prout-bot commented 8 months ago

Seen on leases (merged by @twrichards 8 minutes and 56 seconds ago) Please check your changes!

prout-bot commented 8 months ago

Seen on metadata-editor (merged by @twrichards 9 minutes and 1 second ago) Please check your changes!

prout-bot commented 8 months ago

Seen on kahuna (merged by @twrichards 9 minutes and 52 seconds ago) Please check your changes!