chore: apply code review feedback

Author: castastrophe

migration-roadmap/avatar.md

@@ -8,25 +8,25 @@
 <summary>CSS selectors</summary>
 
 - `.spectrum-Avatar`
+- `.spectrum-Avatar-image`
+- `.spectrum-Avatar-link`
+- `.spectrum-Avatar--size50`
+- `.spectrum-Avatar--size75`
 - `.spectrum-Avatar--size100`
-- `.spectrum-Avatar--size1000`
-- `.spectrum-Avatar--size1100`
-- `.spectrum-Avatar--size1200`
-- `.spectrum-Avatar--size1300`
-- `.spectrum-Avatar--size1400`
-- `.spectrum-Avatar--size1500`
 - `.spectrum-Avatar--size200`
 - `.spectrum-Avatar--size300`
 - `.spectrum-Avatar--size400`
-- `.spectrum-Avatar--size50`
 - `.spectrum-Avatar--size500`
 - `.spectrum-Avatar--size600`
 - `.spectrum-Avatar--size700`
-- `.spectrum-Avatar--size75`
 - `.spectrum-Avatar--size800`
 - `.spectrum-Avatar--size900`
-- `.spectrum-Avatar-image`
-- `.spectrum-Avatar-link`
+- `.spectrum-Avatar--size1000`
+- `.spectrum-Avatar--size1100`
+- `.spectrum-Avatar--size1200`
+- `.spectrum-Avatar--size1300`
+- `.spectrum-Avatar--size1400`
+- `.spectrum-Avatar--size1500`
 - `.spectrum-Avatar.is-disabled`
 - `.spectrum-Avatar.is-focused:not(.is-disabled):after`
 - `.spectrum-Avatar:not(.is-disabled) .spectrum-Avatar-link:focus-visible:after`
@@ -78,7 +78,7 @@ None found for this component.
 ### DOM Structure changes
 
 <details>
-<summary>Spectrum Web Components:</summary>
+<summary>Spectrum Web Components</summary>
 
 ```html
 <!-- With link -->
@@ -93,7 +93,7 @@ None found for this component.
 </details>
 
 <details>
-<summary>Legacy (CSS main branch):</summary>
+<summary>Legacy (CSS main branch)</summary>
 
 ```html
 <!-- With link -->
@@ -122,7 +122,7 @@ None found for this component.
 </details>
 
 <details>
-<summary>Spectrum 2 (CSS spectrum-two branch):</summary>
+<summary>Spectrum 2 (CSS spectrum-two branch)</summary>
 
 ```html
 <!-- With link -->
@@ -187,13 +187,13 @@ No significant structural changes.
 
 ### CSS => SWC implementation gaps
 
-**CSS Features Missing from Web Component:**
+**CSS features missing from Web Component:**
 
 - Disabled state support (`.spectrum-Avatar.is-disabled`)
 - Focus state support (`.spectrum-Avatar.is-focused`)
 - Larger size variants (800, 900, 1000, 1100, 1200, 1300, 1400, 1500)
 
-**Web Component Features Missing from CSS:**
+**Web Component features missing from CSS:**
 None found for this component.
 
 ### CSS Spectrum 2 changes
@@ -202,6 +202,6 @@ No significant structural changes between CSS main and spectrum-two branches. Th
 
 ## Resources
 
-- [CSS migration]()
-- [Spectrum 2 preview]()
-- [React]()
+- [CSS migration](https://github.com/adobe/spectrum-css/pull/3355)
+- [Spectrum 2 preview](https://spectrumcss.z13.web.core.windows.net/pr-2352/index.html?path=/docs/components-avatar--docs)
+- [React](https://react-spectrum.adobe.com/s2/index.html?path=/docs/avatar--docs)

migration-roadmap/opacity-checkerboard.md

@@ -50,7 +50,7 @@ None found for this component. This is a utility component used by other compone
 ### DOM Structure changes
 
 <details>
-<summary>Spectrum Web Components:</summary>
+<summary>Spectrum Web Components</summary>
 
 ```html
 <!-- Used as a utility class in other components -->
@@ -62,7 +62,7 @@ None found for this component. This is a utility component used by other compone
 </details>
 
 <details>
-<summary>Legacy (CSS main branch):</summary>
+<summary>Legacy (CSS main branch)</summary>
 
 ```html
 <div
@@ -78,7 +78,7 @@ None found for this component. This is a utility component used by other compone
 </details>
 
 <details>
-<summary>Spectrum 2 (CSS spectrum-two branch):</summary>
+<summary>Spectrum 2 (CSS spectrum-two branch)</summary>
 
 ```html
 <div
@@ -115,10 +115,10 @@ No significant structural changes.
 
 ### CSS => SWC implementation gaps
 
-**CSS Features Missing from Web Component:**
+**CSS features missing from Web Component:**
 None found for this component. The opacity checkerboard is implemented as a utility class in the Web Components.
 
-**Web Component Features Missing from CSS:**
+**Web Component features missing from CSS:**
 None found for this component.
 
 ### CSS Spectrum 2 changes
@@ -131,6 +131,6 @@ The opacity checkerboard is a utility component that provides a checkerboard pat
 
 ## Resources
 
-- [CSS migration]()
-- [Spectrum 2 preview]()
-- [React]()
+- [CSS migration](https://github.com/adobe/spectrum-css/pull/3394)
+- [Spectrum 2 preview](https://spectrumcss.z13.web.core.windows.net/pr-2352/index.html?path=/docs/components-opacity-checkerboard--docs)
+- [React](https://react-spectrum.adobe.com/s2/index.html?path=/docs/colorslider--docs) (not a standalone component, but styles are used by the color slider)

migration-roadmap/swatch-group.md

@@ -55,7 +55,7 @@
 ### DOM Structure changes
 
 <details>
-<summary>Spectrum Web Components:</summary>
+<summary>Spectrum Web Components</summary>
 
 ```html
 <slot @change="[handleChange]" @slotchange="[manageChange]">
@@ -66,7 +66,7 @@
 </details>
 
 <details>
-<summary>Legacy (CSS main branch):</summary>
+<summary>Legacy (CSS main branch)</summary>
 
 ```html
 <div
@@ -81,7 +81,7 @@
 </details>
 
 <details>
-<summary>Spectrum 2 (CSS spectrum-two branch):</summary>
+<summary>Spectrum 2 (CSS spectrum-two branch)</summary>
 
 ```html
 <div
@@ -120,10 +120,10 @@ No significant structural changes.
 
 ### CSS => SWC implementation gaps
 
-**CSS Features Missing from Web Component:**
+**CSS features missing from Web Component:**
 None found for this component.
 
-**Web Component Features Missing from CSS:**
+**Web Component features missing from CSS:**
 
 - Border attribute support
 - Rounding attribute support
@@ -138,6 +138,6 @@ No significant structural changes between CSS main and spectrum-two branches. Th
 
 ## Resources
 
-- [CSS migration]()
-- [Spectrum 2 preview]()
-- [React]()
+- [CSS migration](https://github.com/adobe/spectrum-css/pull/3677)
+- [Spectrum 2 preview](https://spectrumcss.z13.web.core.windows.net/pr-2352/index.html?path=/docs/components-swatch-group--docs)
+- [React](https://react-spectrum.adobe.com/s2/index.html?path=/docs/colorswatchpicker--docs)

migration-roadmap/swatch.md

@@ -132,7 +132,7 @@ None found for this component.
 ### DOM Structure changes
 
 <details>
-<summary>Spectrum Web Components:</summary>
+<summary>Spectrum Web Components</summary>
 
 ```html
 <div class="opacity-checkerboard fill" style="--spectrum-picked-color: [color]">
@@ -145,7 +145,7 @@ None found for this component.
 </details>
 
 <details>
-<summary>Legacy (CSS main branch):</summary>
+<summary>Legacy (CSS main branch)</summary>
 
 ```html
 <div
@@ -172,7 +172,7 @@ None found for this component.
 </details>
 
 <details>
-<summary>Spectrum 2 (CSS spectrum-two branch):</summary>
+<summary>Spectrum 2 (CSS spectrum-two branch)</summary>
 
 ```html
 <div
@@ -237,7 +237,7 @@ No significant structural changes.
 
 ### CSS => SWC implementation gaps
 
-**CSS Features Missing from Web Component:**
+**CSS features missing from Web Component:**
 
 - Add swatch functionality (`.spectrum-Swatch.is-addSwatch`)
 - Focus state support (`.spectrum-Swatch.is-keyboardFocused`)
@@ -247,15 +247,16 @@ No significant structural changes.
 - Pseudo-element styling (`:before`, `:after` selectors)
 - Focus styling (`:focus-visible`)
 
-**Web Component Features Missing from CSS:**
+**Web Component features missing from CSS:**
+
 None found for this component.
 
 ### CSS Spectrum 2 changes
 
-No significant structural changes between CSS main and spectrum-two branches. The templates are identical, indicating that the swatch component structure remains consistent across Spectrum 2 migration.
+There is a new add swatch functionality in the `spectrum-two` branch.
 
 ## Resources
 
-- [CSS migration]()
-- [Spectrum 2 preview]()
-- [React]()
+- [CSS migration](https://github.com/adobe/spectrum-css/pull/3677)
+- [Spectrum 2 preview](https://spectrumcss.z13.web.core.windows.net/pr-2352/index.html?path=/docs/components-swatch--docs)
+- [React](https://react-spectrum.adobe.com/s2/index.html?path=/docs/colorswatch--docs)

migration-roadmap/thumbnail.md

@@ -9,18 +9,18 @@
 
 - `.spectrum-Thumbnail`
 - `.spectrum-Thumbnail--cover .spectrum-Thumbnail-image`
+- `.spectrum-Thumbnail--size50`
+- `.spectrum-Thumbnail--size75`
 - `.spectrum-Thumbnail--size100`
-- `.spectrum-Thumbnail--size1000`
 - `.spectrum-Thumbnail--size200`
 - `.spectrum-Thumbnail--size300`
 - `.spectrum-Thumbnail--size400`
-- `.spectrum-Thumbnail--size50`
 - `.spectrum-Thumbnail--size500`
 - `.spectrum-Thumbnail--size600`
 - `.spectrum-Thumbnail--size700`
-- `.spectrum-Thumbnail--size75`
 - `.spectrum-Thumbnail--size800`
 - `.spectrum-Thumbnail--size900`
+- `.spectrum-Thumbnail--size1000`
 - `.spectrum-Thumbnail-background`
 - `.spectrum-Thumbnail-image`
 - `.spectrum-Thumbnail-image-wrapper`
@@ -87,7 +87,7 @@
 ### DOM Structure changes
 
 <details>
-<summary>Spectrum Web Components:</summary>
+<summary>Spectrum Web Components</summary>
 
 ```html
 <!-- With background -->
@@ -262,14 +262,14 @@ No significant structural changes.
 
 ### CSS => SWC implementation gaps
 
-**CSS Features Missing from Web Component:**
+**CSS features missing from Web Component:**
 
 - Disabled state support (`.spectrum-Thumbnail.is-disabled`)
 - Focus state support (`.spectrum-Thumbnail.is-focused`)
 - Selected state support (`.spectrum-Thumbnail.is-selected`)
 - Pseudo-element styling (`:before`, `:after` selectors)
 
-**Web Component Features Missing from CSS:**
+**Web Component features missing from CSS:**
 None found for this component.
 
 ### CSS Spectrum 2 changes
@@ -278,6 +278,6 @@ No significant structural changes between CSS main and spectrum-two branches. Th
 
 ## Resources
 
-- [CSS migration]()
-- [Spectrum 2 preview]()
-- [React]()
+- [CSS migration](https://github.com/adobe/spectrum-css/pull/3367)
+- [Spectrum 2 preview](https://spectrumcss.z13.web.core.windows.net/pr-2352/index.html?path=/docs/components-thumbnail--docs)
+- React (no thumbnail component in React Spectrum)