mercuryworkshop-mirror/scramjet
1
0
Fork 0
mirror of https://github.com/MercuryWorkshop/scramjet.git synced 2025-05-14 23:10:02 -04:00
Code Issues Projects Releases Packages Wiki Activity Actions

fix: Improve srcset rewriter to handle descriptors correctly.

Browse source 
The previous implementation failed to properly rewrite srcset attributes
when they contained descriptors (like 1.5x, 2x). Updated the parser to
preserve descriptors while rewriting URLs.
This commit is contained in:
Terry 2024-11-11 08:35:10 +00:00
parent 73357d0172
commit 3015dff77d
1 changed files with 12 additions and 9 deletions
Download patch file Download diff file Expand all files Collapse all files

21
src/shared/rewriters/html.ts
View file

@ -283,17 +283,20 @@ function traverseParsedHtml(
}
export function rewriteSrcset(srcset: string, meta: URLMeta) {
const urls = srcset.split(/ [0-9]+x,? ?/g);
if (!urls) return "";
const sufixes = srcset.match(/ [0-9]+x,? ?/g);
if (!sufixes) return "";
const rewrittenUrls = urls.map((url, i) => {
if (url && sufixes[i]) {
return rewriteUrl(url, meta) + sufixes[i];
}
const sources = srcset.split(",").map((src) => src.trim());
const rewrittenSources = sources.map((source) => {
// Split into URLs and descriptors (if any)
// e.g. url0, url1 1.5x, url2 2x
const [url, ...descriptors] = source.split(/\s+/);
// Rewrite the URLs and keep the descriptors (if any)
const rewrittenUrl = rewriteUrl(url.trim(), meta);
return descriptors.length > 0
? `${rewrittenUrl} ${descriptors.join(" ")}`
: rewrittenUrl;
});
return rewrittenUrls.join("");
return rewrittenSources.join(", ");
}
// function base64ToBytes(base64) {