Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: SingleMsgDS focal point updates #508

Open
wants to merge 7 commits into
base: master
Choose a base branch
from

Conversation

AndyEPhipps
Copy link
Contributor

@AndyEPhipps AndyEPhipps commented Oct 11, 2022

PR description

What is it doing?

This will place a object-fit position CSS rule on an SingleMessageDS image when supplied with the following details via the new CMS fields:

  • raw image dimensions from the new Focal Point Image field (keeping the original image field for now to allow backwards compatibility)
  • the focal point associated with this image, set manually in the CMS

Why is this required?

Request from Design/Content; the current styles will just centre the image, and depending on the image and breakpoint, it can sometimes look a bit crap; this gives us the ability to better position the image on a case-by-case basis.

link to Jira ticket:

https://comicrelief.atlassian.net/browse/ENG-1737

@AndyEPhipps AndyEPhipps changed the title Focal point props feat: SingleMsgDS focal point updates Oct 11, 2022
@AndyEPhipps AndyEPhipps self-assigned this Oct 13, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant