Skip to content
This repository has been archived by the owner on Feb 28, 2022. It is now read-only.

fix: respect image hashes with dimensions #1134

Merged
merged 1 commit into from
Aug 24, 2021
Merged

Conversation

tripodsan
Copy link
Contributor

with the inclusion of the image dimensions, we should remove them when rewriting the image urls to the media-urls, until the helix-pages picture handling is adjusted accordingly.

@tripodsan tripodsan requested a review from rofe August 23, 2021 09:01
@codecov
Copy link

codecov bot commented Aug 23, 2021

Codecov Report

Merging #1134 (96f0ed7) into main (f952b87) will not change coverage.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff            @@
##              main     #1134   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files           60        60           
  Lines         1623      1623           
=========================================
  Hits          1623      1623           
Impacted Files Coverage Δ
src/html/rewrite-blob-images.js 100.00% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update f952b87...96f0ed7. Read the comment docs.

@tripodsan tripodsan merged commit 9390c93 into main Aug 24, 2021
@tripodsan tripodsan deleted the respect-dimensions branch August 24, 2021 07:59
adobe-bot pushed a commit that referenced this pull request Aug 24, 2021
## [14.0.34](v14.0.33...v14.0.34) (2021-08-24)

### Bug Fixes

* respect image hashes with dimensions ([#1134](#1134)) ([9390c93](9390c93))
@adobe-bot
Copy link

🎉 This PR is included in version 14.0.34 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants