mainmatter / mainmatter.com

The source code for https://mainmatter.com
16 stars 9 forks source link

turn off proseWrap #2492

Closed mansona closed 2 months ago

mansona commented 4 months ago

Previously we have had prosewrap turned on to make sure there is consistency in the line width for markdown files. This changes the prosewrap to have a value of never to bring the consistency in the opposite way.

Reasons for this change:

Personal reasons for this change:

In most cases, my dyslexia doesn't affect my work; this is mainly because I can configure the tools I use to work most effectively for me. When line breaks are forced in markdown documents I can't edit my local settings to correct this, thus forcing any processes to be significantly harder for me. That plus the extra cognitive load of diffs like this make blog writing/reviewing a thoroughly unpleasant experience for me.

I understand that a PR that changes every file in the repo is not easiest thing to consider but I would like us not to dismiss it outright because it's difficult. I don't mind a PR like this being rejected as long as it's seriously considered and we actively decide to reject it 👍

mansona commented 3 months ago

In case anyone is wondering why I'm continuously force pushing: as this changes every Markdown in the website I need to rebase it every time anything is merged. I'm rebasing any time I notice anyone has pushed because I'm eager to keep this in a mergeable state 👍