monero-project / monero-site

https://getmonero.org
BSD 3-Clause "New" or "Revised" License
275 stars 384 forks source link

_includes/header.html: liquid whitespace control #2322

Open plowsof opened 2 months ago

plowsof commented 2 months ago

we're putting whitespace into the raw html files due to not using any whitespace control in the liquid script which decides what locale to add to the homepage url.

This can effect plugins which parse/edit urls such as jekyll-loading-lazy which we dont actually need but its just bad practice and could effect other plugins in the future.

if we inspect the monero logo / homepage shortcut in the header at getmonero.org we see: Screenshot from 2024-07-16 10-13-09

with this change: Screenshot from 2024-07-16 10-29-01

netlify[bot] commented 2 months ago

Deploy Preview for barolo-time-757cf9 ready!

Built without sensitive environment variables

Name Link
Latest commit b7e56a5b00147e64aacc9756739458a505441ea0
Latest deploy log https://app.netlify.com/sites/barolo-time-757cf9/deploys/66963f22dffd030008933105
Deploy Preview https://deploy-preview-2322--barolo-time-757cf9.netlify.app
Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.