Skip to content

Conversation

@dspace-bot
Copy link
Contributor

Port of #4636 by @alanorth to dspace-8_x.

For some reason our version of this image is twice the size of the
original (~860KiB vs 1.6MiB). To make matters worse, the dimensions
of the image are 4777x3166 pixels, but we display it at 2000 pixels
wide.

This regenerates the images based on the original and adjusts the
dimensions to be more appropriate. Generated using ImageMagick 7
on Linux:

$ magick pexels-inspiredimages-132477.jpg -resize 2000x src/themes/dspace/assets/images/banner.jpg
$ magick pexels-inspiredimages-132477.jpg -resize 2000x src/themes/dspace/assets/images/banner.webp
$ magick pexels-inspiredimages-132477.jpg -resize 1200x src/themes/dspace/assets/images/banner-half.jpg
$ magick pexels-inspiredimages-132477.jpg -resize 1200x src/themes/dspace/assets/images/banner-half.webp

See: https://www.pexels.com/photo/clear-water-drops-132477/
(cherry picked from commit d9e8fea)
@dspace-bot dspace-bot added 1 APPROVAL pull request only requires a single approval to merge performance / caching Related to performance, caching or embedded objects labels Sep 10, 2025
@tdonohue tdonohue added this to the 8.3 milestone Sep 10, 2025
@tdonohue tdonohue merged commit b67a703 into dspace-8_x Sep 10, 2025
15 checks passed
@tdonohue tdonohue deleted the backport-4636-to-dspace-8_x branch September 10, 2025 19:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

1 APPROVAL pull request only requires a single approval to merge performance / caching Related to performance, caching or embedded objects

Projects

Development

Successfully merging this pull request may close these issues.

4 participants