snshn
commented
5 years ago Hey! I really tried!
- 1:
<pre>created some issues, I believe that was the original approach, but then I had to change it, don't remember now why, it's been years. I'll comment my code better next time to keep those notes. Possibly it was related to SEO but not sure, could try to use<pre>again and see what happens. 😶 - 2 & 3:
- 4: The HTML delta was too big, hundreds of files. I agree that the minified CSS file shouldn't be included in the PR. It has to be in the tree since it's a GitHub Pages repo, but it should be changed during build time. I'll switch it to use Sass in a separate branch to avoid these inconsistencies. Maybe the approach is wrong, e.g. the website should be in one repo, and the build-time tools and "source" database should be in a separate one...
This wraps everything below a separating line into a DIV to make it easier to visually separate song text from metadata