Improve project's CSS

[julieg18]

• Add stylelint into pre-commit hook (webview/.lintstagedrc.js)
• Enable stylelint camelCase selector rules (Add Stylelint #3736 (comment))
• Move stylelint packages into webview/node_modules instead of the root /node_modules (Add Stylelint #3736 (comment))
• Remove table/styles disabled comments (Add Stylelint #3736 (comment)) - won't do

Followup to #3736

[mattseddon]

A couple of points I would add:

• Follow up on this: Add Stylelint #3736 (comment)
• Add stylelint into pre-commit hook (webview/.lintstagedrc.js)

[julieg18]

Follow up on this: #3736 (comment)

Yes, that's the second third item on our task list, "Move stylelint packages into webview/node_modules instead of the root /node_modules". Though I see I've pasted the wrong comment link, I'll update it.

Add stylelint into pre-commit hook (webview/.lintstagedrc.js)

Good idea!

[mattseddon]

I actually meant this:

I tried to install stylelint packages inside of webview via the stylelint.stylelintPath config option, but stylelint kept getting installed inside the root for some reason, even after moving packages inside of webview/package.json, deleting node_modules, and starting out of a fresh install. I even tried to revert yarn.lock to a previous commit and fresh install the packages but stylelint still appeared inside of /node_modules instead of /webview/node_modules. Kept the packages installed in the root for now but can revisit this later

[julieg18]

I actually meant this:

I tried to install stylelint packages inside of webview via the stylelint.stylelintPath config option, but stylelint kept getting installed inside the root for some reason, even after moving packages inside of webview/package.json, deleting node_modules, and starting out of a fresh install. I even tried to revert yarn.lock to a previous commit and fresh install the packages but stylelint still appeared inside of /node_modules instead of /webview/node_modules. Kept the packages installed in the root for now but can revisit this later

...Yes, that is the exact comment I'm referring too 🤔 The packages should be installed inside of /webview instead of the root since they're only used in the webview but I was having issues moving them... Is there something I'm misunderstanding?

[mattseddon]

I got as far as

Yes, that's the second item on our task list

which is

Enable stylelint camelCase selector rules (#3736 (comment))

but the rest of the sentence clears everything up. Sorry.

[julieg18]

I got as far as

Yes, that's the second item on our task list

Whoops! I think it was the second item at the time when I wrote the comment... Or I just made a typo 😄