Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Hey @matuzo
Fixes #74
In this PR
The bulk of this PR is to do with logo usage
logo.txt
XHR script with an inlined SVG in the HTMLpentagramm.svg
andpentagramm-light.svg
versions with the new structure.Notes
<use>
the pentacle<circle>
), since Chromium/Webkit browsers don't allow for external URI<mask>
s.pentagramm(-light).svg
with inline versions, and therefore re-using the SVG sprite.logo.svg
in the Web Manifest, and Safari pinned tab mask. I'll leave that for PWA icon fixes.Results
The logo improvments can be seen in the comments on #74
As for site performance, rudimentary testing on my local environment yielded:
125kB
(before)102kB
(after)21kB
(before)56kB
(after)The HTML payload is now
11kB
g-zipped, which keeps us below the14kB
threshhold.Waddayathink?
-S