-
-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
Fix sass quoted string interpolation for hugo 0.110.0 #1566
Conversation
I believe these wrapping quotes also need to be applied here: |
Ok, I added the same change to |
@mdo can you check this PR and the related linked issues? Unsure if this the fix but judging by the issues there might be a real problem somewhere :) |
…back after it was removed by ea46c09
Please don't touch the dist file and don't revert our changes. |
@XhmikosR unfortunately you removed the fix in ea46c09, so I added it back. We need the quotes around $bootstrap-icons-font-file: "#{$bootstrap-icons-font-dir}/#{$bootstrap-icons-font}" !default; Because the sass compiler that hugo uses does not interpolate the quotes from If we do not add the quotes around
and the extra quotes break the url. |
No we don't need it manually. The file is auto-generated so just revert the change like I did |
Ok. I'll revert it. |
Cool, I ran the build process locally and I see the fix changes:
|
if
$bootstrap-icons-font-file: "#{$bootstrap-icons-font-dir}/#{$bootstrap-icons-font}" !default;
is unquoted then it results insrc:url('./fonts/"bootstrap-icons".woff2?24e3eb84d0bcaf83d77f904c78ac1f47'
with the quotes in hugo 0.110.0 extended.fixes #1378, fixes #1538, fixes #1574