taho.xyz
taho.xyz copied to clipboard
Minor README Update + Create Issue : updated to tallyho.org from cash, added tip for local env
This PR updates the title from Tallyho.cash to the currently-known one, Tallyho.org.
Furthermore, while reproducing the dev environment on my local setup, I faced a few issues in Windows
. I found it to be working on Linux
machine but had errors that showed compatibility issues with node@18
and suggested downgrading to node@16
, which worked for me.
I also have some minor tweaks for website optimizations, if needed I can add them as an issue and work on it as well.
A few of them are:
-
alt
attributes for most of theimg
tags is missing, we can add them for SEO optimizations. We should also explicitly addwidth
andheight
to the images as well. - We can also transform the images from
PNG
to web-friendly formats such asWebP
andAVIF
, resulting in tremendous size and speed savings as seen below on pagespeed:
Thanks for the contribution @abhiram11! Any chance you can merge man
into your branch?
@Shadowfiend can you take a peek at this when you get a minute?
Hi @mhluongo , I am new to git and I did not understand how to "merge main into your branch". Were you suggesting git merge main
, git rebase main
, or something else? I tried the above two though.
Update 1 : I have merged main into the branch, hope it has updated as required : )
Thanks!