updated the largest size files I found in the docs and removed some t…#1367
updated the largest size files I found in the docs and removed some t…#1367Socialhappiness wants to merge 2 commits intomasterfrom
Conversation
…ext on the clear cache and cookies article while I replaced the gif with images.
jacobgkau
left a comment
There was a problem hiding this comment.
The old PDFs did not have titles. The new ones have titles that say Untitled, which could be annoying:
It looks like some margins also got added. What tool did you use to compress those?
For PDFs, another solution instead of (or in addition to) shrinking them would be to stick them in the company Dropbox or Google Drive and generate public sharing links. They're already just links, so there's no reason they need to be hosted on the same infrastructure as the website itself.
|
Replacing the gifs with photos and more descriptive text is a good change. |
|
I can't remember what tool I tried, feels like an eternity ago but I think hosting on Drive and linking was a good call and I believe this latest commit achieves that |
thomas-zimmerman
left a comment
There was a problem hiding this comment.
Live build of the page looks good. Used a link checker and all of the links come back good:
linkchecker --check-extern http://localhost:3000/articles/guides
Statistics:
Downloaded: 147.50KB.
Content types: 1 image, 94 text, 0 video, 0 audio, 10 application, 0 mail and 0 other.
URL lengths: min=20, max=115, avg=68.
That's it. 105 links in 119 URLs checked. 14 warnings found. 0 errors found.
Stopped checking at 2026-04-03 15:32:05-006 (36 seconds)
All of the warnings are for links out to the rest of the S76 site that have 301 redirects to the "real" site, not on the NPM served version.
…ext on the clear cache and cookies article while I replaced the gif with images.