Update table classes for consistency

[Jeremy-Walton]

Why?

The Table class modifiers were not consistent in their naming with other optics classes. This adds more consistent names while retaining backwards compatibility.

What Changed

• Update table density and layout names
• Update docs to reflect
• Retain old names for backwards compatibility

Quality Assurance

• Have you tagged the PR with the correct labels?
• Have you validated the changes?
  • Have you run linters? (yarn sanity-check)
  • Have you run prettier?
  • Have you tried building the css?
  • Have you tried building storybook?
• Have you updated any usage of changed tokens?
• [ ] Did you add a component?
  • [ ] Have you added it to the dependency graph?
  • [ ] Have you added it to the docs?
• Did you update a component?
  • [ ] Have you updated the dependency graph?
  • Have you updated the docs?
• Do you need to update the package version?
  • Did you update the example pages in .storybook/assets?
• [ ] Were any changes made to the top level optics.css file?
  • [ ] Were those changes reflected in the other top level files?

[dallasbpeters]

Should the old classes be marked as deprecated? It would be nice if linters and other tools could pick up on the need to update the class.

[Jeremy-Walton]

@dallasbpeters That would be nice but we don't have an Optics linter at the moment. I did make https://github.com/Jeremy-Walton/stylelint-bem and could totally see making something similar for Optics. We would just need to talk through what we want the rules to be.