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

UHF-10191: Image style refactor #786

Merged
merged 32 commits into from
Aug 30, 2024
Merged

UHF-10191: Image style refactor #786

merged 32 commits into from
Aug 30, 2024

Conversation

teroelonen
Copy link
Contributor

@teroelonen teroelonen commented Jul 22, 2024

UHF-10191

What was done

  • Add new image styles and responsive image styles.
  • Add DEPRECATED to old image style and responsive image style labels.
  • Add new media view displays.
  • Change the image processing for the React searches to print out derivates instead of just one image style.

How to install

  • Check installing instructions from the hdbt PR.

How to test

  • Check testing instructions from the hdbt PR.
  • Check that code follows our standards

Continuous documentation

  • This feature has been documented/the documentation has been updated
  • This change doesn't require updates to the documentation

Other PRs

…ome of the old responsive image styles that will be still used after the refactoring
…to work similar to current hero bottom responsive style to avoid image issues
…s to work similar to current hero diagonal responsive style to avoid image issues
…media view display to use the new responsive image style
…ze, add media view display and TPR unit view display for unit contact card, change tpr unit teaser with image to use card media view display
Copy link
Contributor

@khalima khalima left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Excellent! 🦜

Copy link

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.

2 participants