Implement option to support adding spaces between CSS selectors

[kcamsanc]

Description

• Source branch in your fork has meaningful name (not main)

Fixes Issue:
Fixes #1862 - Added a new option called selector-separator.
Added unit tests and updated the README/CLI options accordingly.

Before Merge Checklist

These items can be completed after PR is created.

(Check any items that are not applicable (NA) for this PR)

• JavaScript implementation
• Python implementation (NA if HTML beautifier)
• Added Tests to data file(s)
• Added command-line option(s) (NA if
• README.md documents new feature/option(s)

[bitwiseman]

Thanks for working on this!

You'll need to resolve conflicts.

Remove selector-separator-newline from the documented options and note it is deprecated in code comments.

I think I'd rather have the option be named selector-separator.
Default value is "default". If the value is set to null or "", the value will be treated as "default".

• If set to "default" and selector-separator-newline is false, then add a space between selectors
• If set to "default" and selector-separator-newline is true, then add a newline between selectors
• If set to anything other than "default", then ignore selector-separator-newline and:
  • If set to "none", then remove all whitespace between selectors.
  • If set to "newline", then add a newline between selectors.
  • If set to "space", then add a space between selectors.
  • For any other value, throw error.

[kcamsanc]

Sounds good, I'll address this soon.

If set to "default" and selector-separator-newline is false, then add a space between selectors
If set to "default" and selector-separator-newline is true, then add a space between selectors

To clarify, if selector-separator is set to "default", we want to add a space between selectors regardless of the value of selector-separator-newline?

[bitwiseman]

No, sorry. I've updated my comment. "default" is the same as current behavior.

[kcamsanc]

Okay, just updated my changes based on your comments, including updating the README and unit tests. Let me know how it looks.

[bitwiseman]

Looks like you have some conflicts to resolve before we can run the build.

[kcamsanc]

Just resolved those conflicts👍🏼