Closed sigzero closed 1 year ago
The whole site is hard to read, especially for people with eyes older than 20. I know that designers think low-contrast text is somehow a good thing and aesthetically pleasing, but from a practical point of view, they're wrong.
I agree. PRs welcome.
@wardi the change seems simple enough, and looking at the screenshot, it looks good to me. I am inclined to merge it, sounds good?
btw, I also created https://github.com/wardi/jsonlines/issues/67 because I think you do the deployment manually (I think you mentioned that to me at some point), so I am not sure if just merging the changes would have any impact if you don't deploy after.
@sp4ce go for it. The site is html at the moment so there is no deployment process other than pushing changes.
The examples have red on black. Very hard to read. Either just use HTML formatting or change the theme for the pictures. Please.