Tabset padding

[Blackbaud-ToddRoberts]

#1166

[codecov-io]

Codecov Report

Merging #1167 into master will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff           @@
##           master   #1167   +/-   ##
======================================
  Coverage     100%    100%           
======================================
  Files         355     355           
  Lines        6592    6592           
  Branches      844     844           
======================================
  Hits         6592    6592

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update a982def...82cdf86. Read the comment docs.

[Blackbaud-SteveBrush]

@Blackbaud-PaulCrowder This branch will add padding to the tab component and requires that we delete some screenshots to allow for the tests to pass. This brings up the question: what constitutes a "visual" breaking change? Would simple adjustments to padding be a candidate for a major release?

[Blackbaud-PaulCrowder]

It's a good question and one that might have a slightly different answer than with SKY UX 1. With SKY UX 1 we knew of consumers doing visual regression tests that would break if we introduced a visual change, so we tended to bundle visual changes up into one (then minor to indicate a breaking change) release. I don't know that the same situation exists for consumers of SKY UX 2, so I'm OK with making these minor visual changes in minor version updates until we get feedback that it's problematic.