chore(menu): remove variable fallbacks

castastrophe · castastrophe · commit da08c0f0c01d · 2025-04-17T10:33:30.000-04:00
diff --git a/.changeset/stale-candies-camp.md b/.changeset/stale-candies-camp.md
@@ -1,6 +1,8 @@
 ---
-'@spectrum-web-components/tooltip': patch
+'@spectrum-web-components/menu': patch
+'@spectrum-web-components/styles': patch
 '@spectrum-web-components/toast': patch
+'@spectrum-web-components/tooltip': patch
 ---
 
 Remove unnecessary system theme references to reduce complexity for components that don't need the additional mapping layer.
diff --git a/packages/menu/src/spectrum-checkmark.css b/packages/menu/src/spectrum-checkmark.css
@@ -32,9 +32,7 @@ governing permissions and limitations under the License.
 }
 
 .spectrum-Menu-back {
-    padding-inline: 0 var(--mod-menu-back-padding-inline-end, var(--spectrum-menu-item-label-inline-edge-to-content));
     padding-inline: var(--mod-menu-back-padding-inline-start, 0) var(--mod-menu-back-padding-inline-end, var(--spectrum-menu-item-label-inline-edge-to-content));
-    padding-block: 0;
     padding-block: var(--mod-menu-back-padding-block-start, 0) var(--mod-menu-back-padding-block-end, 0);
     flex-flow: wrap;
     align-items: center;
diff --git a/packages/menu/src/spectrum-chevron.css b/packages/menu/src/spectrum-chevron.css
@@ -27,9 +27,7 @@ governing permissions and limitations under the License.
 }
 
 .spectrum-Menu-back {
-    padding-inline: 0 var(--mod-menu-back-padding-inline-end, var(--spectrum-menu-item-label-inline-edge-to-content));
     padding-inline: var(--mod-menu-back-padding-inline-start, 0) var(--mod-menu-back-padding-inline-end, var(--spectrum-menu-item-label-inline-edge-to-content));
-    padding-block: 0;
     padding-block: var(--mod-menu-back-padding-block-start, 0) var(--mod-menu-back-padding-block-end, 0);
     flex-flow: wrap;
     align-items: center;
diff --git a/packages/menu/src/spectrum-menu-divider.css b/packages/menu/src/spectrum-menu-divider.css
@@ -27,9 +27,7 @@ governing permissions and limitations under the License.
 }
 
 .spectrum-Menu-back {
-    padding-inline: 0 var(--mod-menu-back-padding-inline-end, var(--spectrum-menu-item-label-inline-edge-to-content));
     padding-inline: var(--mod-menu-back-padding-inline-start, 0) var(--mod-menu-back-padding-inline-end, var(--spectrum-menu-item-label-inline-edge-to-content));
-    padding-block: 0;
     padding-block: var(--mod-menu-back-padding-block-start, 0) var(--mod-menu-back-padding-block-end, 0);
     flex-flow: wrap;
     align-items: center;
diff --git a/packages/menu/src/spectrum-menu-item.css b/packages/menu/src/spectrum-menu-item.css
@@ -355,9 +355,7 @@ governing permissions and limitations under the License.
 }
 
 .spectrum-Menu-back {
-    padding-inline: 0 var(--mod-menu-back-padding-inline-end, var(--spectrum-menu-item-label-inline-edge-to-content));
     padding-inline: var(--mod-menu-back-padding-inline-start, 0) var(--mod-menu-back-padding-inline-end, var(--spectrum-menu-item-label-inline-edge-to-content));
-    padding-block: 0;
     padding-block: var(--mod-menu-back-padding-block-start, 0) var(--mod-menu-back-padding-block-end, 0);
     flex-flow: wrap;
     align-items: center;
diff --git a/packages/menu/src/spectrum-menu-sectionHeading.css b/packages/menu/src/spectrum-menu-sectionHeading.css
@@ -32,9 +32,7 @@ governing permissions and limitations under the License.
 }
 
 .spectrum-Menu-back {
-    padding-inline: 0 var(--mod-menu-back-padding-inline-end, var(--spectrum-menu-item-label-inline-edge-to-content));
     padding-inline: var(--mod-menu-back-padding-inline-start, 0) var(--mod-menu-back-padding-inline-end, var(--spectrum-menu-item-label-inline-edge-to-content));
-    padding-block: 0;
     padding-block: var(--mod-menu-back-padding-block-start, 0) var(--mod-menu-back-padding-block-end, 0);
     flex-flow: wrap;
     align-items: center;
diff --git a/packages/menu/src/spectrum-menu.css b/packages/menu/src/spectrum-menu.css
@@ -166,7 +166,6 @@ governing permissions and limitations under the License.
 }
 
 :host {
-    inline-size: auto;
     inline-size: var(--mod-menu-inline-size, auto);
     box-sizing: border-box;
     margin: 0;
@@ -222,9 +221,7 @@ governing permissions and limitations under the License.
 }
 
 .spectrum-Menu-back {
-    padding-inline: 0 var(--mod-menu-back-padding-inline-end, var(--spectrum-menu-item-label-inline-edge-to-content));
     padding-inline: var(--mod-menu-back-padding-inline-start, 0) var(--mod-menu-back-padding-inline-end, var(--spectrum-menu-item-label-inline-edge-to-content));
-    padding-block: 0;
     padding-block: var(--mod-menu-back-padding-block-start, 0) var(--mod-menu-back-padding-block-end, 0);
     flex-flow: wrap;
     align-items: center;
diff --git a/yarn.lock b/yarn.lock

Original file line number	Diff line number	Diff line change
`@@ -32,9 +32,7 @@ governing permissions and limitations under the License.`
`32`	`32`	`}`
`33`	`33`
`34`	`34`	`.spectrum-Menu-back {`
`35`		`- padding-inline: 0 var(--mod-menu-back-padding-inline-end, var(--spectrum-menu-item-label-inline-edge-to-content));`
`36`	`35`	`padding-inline: var(--mod-menu-back-padding-inline-start, 0) var(--mod-menu-back-padding-inline-end, var(--spectrum-menu-item-label-inline-edge-to-content));`
`37`		`- padding-block: 0;`
`38`	`36`	`padding-block: var(--mod-menu-back-padding-block-start, 0) var(--mod-menu-back-padding-block-end, 0);`
`39`	`37`	`flex-flow: wrap;`
`40`	`38`	`align-items: center;`
Original file line number	Diff line number	Diff line change
`@@ -27,9 +27,7 @@ governing permissions and limitations under the License.`
`27`	`27`	`}`
`28`	`28`
`29`	`29`	`.spectrum-Menu-back {`
`30`		`- padding-inline: 0 var(--mod-menu-back-padding-inline-end, var(--spectrum-menu-item-label-inline-edge-to-content));`
`31`	`30`	`padding-inline: var(--mod-menu-back-padding-inline-start, 0) var(--mod-menu-back-padding-inline-end, var(--spectrum-menu-item-label-inline-edge-to-content));`
`32`		`- padding-block: 0;`
`33`	`31`	`padding-block: var(--mod-menu-back-padding-block-start, 0) var(--mod-menu-back-padding-block-end, 0);`
`34`	`32`	`flex-flow: wrap;`
`35`	`33`	`align-items: center;`
Original file line number	Diff line number	Diff line change
`@@ -355,9 +355,7 @@ governing permissions and limitations under the License.`
`355`	`355`	`}`
`356`	`356`
`357`	`357`	`.spectrum-Menu-back {`
`358`		`- padding-inline: 0 var(--mod-menu-back-padding-inline-end, var(--spectrum-menu-item-label-inline-edge-to-content));`
`359`	`358`	`padding-inline: var(--mod-menu-back-padding-inline-start, 0) var(--mod-menu-back-padding-inline-end, var(--spectrum-menu-item-label-inline-edge-to-content));`
`360`		`- padding-block: 0;`
`361`	`359`	`padding-block: var(--mod-menu-back-padding-block-start, 0) var(--mod-menu-back-padding-block-end, 0);`
`362`	`360`	`flex-flow: wrap;`
`363`	`361`	`align-items: center;`