Browser-compat-data: Single/double spacing in notes?

Created on 16 Sep 2019  路  3Comments  路  Source: mdn/browser-compat-data

With all the styling we're incorporating, I'm wondering if we should enforce single or double spacing in between sentences in notes, to maintain a more consistent writing style. Since double-spaces get rendered as single spaces by default in browsers anyways, I'm thinking we should stick to single spacing, but I figured I'd open up a discussion here!

question

Most helpful comment

As a reviewer, I really dislike having to make the call about whether to say something to a PR author about double spacing. I don't like double spacing as a matter of preference and it's not typical in this repo, so I feel like I ought to say something. But it's a nitpicky thing that's distracting and unwelcoming to new contributors, so I don't always do it. Enforcement deals with the inconsistency and puts the problem of saying something about it on a script. 馃憤

All 3 comments

I'm not sure it's necessary to enforce this too rigidly.

As a reviewer, I really dislike having to make the call about whether to say something to a PR author about double spacing. I don't like double spacing as a matter of preference and it's not typical in this repo, so I feel like I ought to say something. But it's a nitpicky thing that's distracting and unwelcoming to new contributors, so I don't always do it. Enforcement deals with the inconsistency and puts the problem of saying something about it on a script. 馃憤

I opened this issue as I had an exact instance of what @ddbeck is referring to. A PR I recently reviewed introduced a change to a note that provided a double space (and nothing else on that line). I decided to merge as-is without requesting a change as I felt it was not particularly reasonable to ask them to revert it, since there鈥檚 no written rule on it. If we can throw it into a linter, it can help reduce these issues in the future!

Was this page helpful?
0 / 5 - 0 ratings

Related issues

vinyldarkscratch picture vinyldarkscratch  路  3Comments

ddbeck picture ddbeck  路  3Comments

ExE-Boss picture ExE-Boss  路  3Comments

ddbeck picture ddbeck  路  3Comments

MendyBerger picture MendyBerger  路  3Comments