Skip to content

Commit cf4fc1a

Browse files
authored
fix: minor linting updates (#4113)
1 parent 1e8d49b commit cf4fc1a

File tree

28 files changed

+151
-147
lines changed

28 files changed

+151
-147
lines changed

.changeset/ready-apes-pull.md

Lines changed: 33 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,33 @@
1+
---
2+
"@spectrum-css/coachindicator": patch
3+
"@spectrum-css/alertbanner": patch
4+
"@spectrum-css/colorhandle": patch
5+
"@spectrum-css/colorslider": patch
6+
"@spectrum-css/inlinealert": patch
7+
"@spectrum-css/breadcrumb": patch
8+
"@spectrum-css/colorwheel": patch
9+
"@spectrum-css/datepicker": patch
10+
"@spectrum-css/assetcard": patch
11+
"@spectrum-css/thumbnail": patch
12+
"@spectrum-css/calendar": patch
13+
"@spectrum-css/combobox": patch
14+
"@spectrum-css/dropzone": patch
15+
"@spectrum-css/treeview": patch
16+
"@spectrum-css/underlay": patch
17+
"@spectrum-css/sidenav": patch
18+
"@spectrum-css/stepper": patch
19+
"@spectrum-css/card": patch
20+
"@spectrum-css/dial": patch
21+
"@spectrum-css/page": patch
22+
"@spectrum-css/well": patch
23+
---
24+
25+
## Minor linting fixes
26+
27+
- Removed unused custom properties across multiple components
28+
- Fixed color function notation by converting `rgba()` to `rgb()` where appropriate
29+
- Removed duplicate custom property declarations
30+
- Fixed deprecated CSS properties (e.g., `word-wrap``overflow-wrap`)
31+
- Removed unnecessary stylelint-disable comments
32+
33+
_No visual or functional changes to components._

components/alertbanner/dist/metadata.json

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -61,7 +61,6 @@
6161
"--spectrum-alert-banner-max-inline-size",
6262
"--spectrum-alert-banner-min-height",
6363
"--spectrum-alert-banner-minimum-height",
64-
"--spectrum-alert-banner-neutral-background",
6564
"--spectrum-alert-banner-text-margin-block-end",
6665
"--spectrum-alert-banner-text-margin-block-start",
6766
"--spectrum-alert-banner-text-to-button-horizontal",

components/alertbanner/index.css

Lines changed: 3 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -12,25 +12,23 @@
1212
*/
1313

1414
.spectrum-AlertBanner {
15-
--spectrum-alert-banner-neutral-background: var(--spectrum-neutral-subdued-background-color-default);
16-
1715
--spectrum-alert-banner-min-height: var(--spectrum-alert-banner-minimum-height);
1816
--spectrum-alert-banner-max-inline-size: var(--spectrum-alert-banner-width);
1917
--spectrum-alert-banner-inline-size: auto;
2018

2119
--spectrum-alert-banner-font-size: var(--spectrum-font-size-100);
2220
--spectrum-alert-banner-line-height: var(--spectrum-line-height-100);
2321
--spectrum-alert-banner-font-family: var(--spectrum-sans-font-family-stack);
24-
--spectrum-alert-banner-icon-size: var(--spectrum-workflow-icon-size-100); /* stylelint-disable-line spectrum-tools/no-unused-custom-properties -- used in passthrough */
22+
--spectrum-alert-banner-icon-size: var(--spectrum-workflow-icon-size-100);
2523

2624
/* spacing */
2725
--spectrum-alert-banner-icon-to-text: var(--spectrum-text-to-visual-300);
2826
--spectrum-alert-banner-inline-start-to-content: var(--spectrum-spacing-300);
2927
--spectrum-alert-banner-inline-end-to-content: var(--spectrum-spacing-300);
3028
--spectrum-alert-banner-text-to-button-horizontal: var(--spectrum-spacing-300);
3129
--spectrum-alert-banner-block-edge-to-button: var(--spectrum-spacing-200);
32-
--spectrum-alert-banner-close-button-to-content: var(--spectrum-spacing-300); /* stylelint-disable-line spectrum-tools/no-unused-custom-properties -- used in passthrough */
33-
--spectrum-alert-banner-close-button-to-inline-end: var(--spectrum-alert-banner-close-button-spacing); /* stylelint-disable-line spectrum-tools/no-unused-custom-properties -- used in passthrough */
30+
--spectrum-alert-banner-close-button-to-content: var(--spectrum-spacing-300);
31+
--spectrum-alert-banner-close-button-to-inline-end: var(--spectrum-alert-banner-close-button-spacing);
3432

3533
/* Top and bottom margins for AlertBanner-text. Subtracts the vertical spacing that is already included in the
3634
padding-block of its parent AlertBanner-body element. */

components/assetcard/index.css

Lines changed: 2 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -13,15 +13,11 @@
1313

1414
/* outer container, unstyled */
1515
.spectrum-AssetCard {
16-
--spectrum-assetcard-overlay-background-color: rgba(27 127 245 / 10%);
1716
--spectrum-assetcard-background-color: var(--spectrum-gray-75);
18-
--spectrum-assetcard-selectionindicator-background-color-default: rgba(var(--spectrum-gray-75-rgb), 0.9);
19-
--spectrum-assetcard-selectionindicator-box-shadow-color: var(--spectrum-transparent-black-300);
2017

2118
/* todo: this isn't quite the size from the XD file as 232px is not a size token, so we use 224px */
2219
--spectrum-assetcard-asset-size: 224px;
23-
--spectrum-assetcard-background-color: var(--spectrum-gray-75);
24-
--spectrum-assetcard-overlay-background-color: rgba(27 127 245 / 10%);
20+
--spectrum-assetcard-overlay-background-color: rgb(27 127 245 / 10%);
2521
--spectrum-assetcard-asset-animation-duration: var(--spectrum-animation-duration-100);
2622
--spectrum-assetcard-asset-container-border-size: 1px;
2723
--spectrum-assetcard-header-margin-block-start: var(--spectrum-spacing-300);
@@ -42,7 +38,7 @@
4238
--spectrum-assetcard-selectionindicator-color: var(--spectrum-white);
4339
--spectrum-assetcard-selectionindicator-font-weight: var(--spectrum-bold-font-weight);
4440
--spectrum-assetcard-selectionindicator-font-size: var(--spectrum-font-size-400);
45-
--spectrum-assetcard-selectionindicator-background-color-default: rgba(var(--spectrum-gray-75-rgb), 0.9);
41+
--spectrum-assetcard-selectionindicator-background-color-default: rgb(var(--spectrum-gray-75-rgb), 0.9);
4642
--spectrum-assetcard-selectionindicator-box-shadow-color: var(--spectrum-transparent-black-300);
4743

4844
/* title */

components/breadcrumb/index.css

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -61,7 +61,7 @@
6161
--spectrum-breadcrumbs-action-button-color: var(--spectrum-neutral-subdued-content-color-default);
6262
--spectrum-breadcrumbs-action-button-color-disabled: var(--spectrum-disabled-content-color);
6363
--spectrum-breadcrumbs-focus-indicator-color: var(--spectrum-focus-indicator-color);
64-
--spectrum-breadcrumbs-item-dragged-background: rgba(var(--spectrum-drop-zone-background-color-rgb), var(--spectrum-drop-zone-background-color-opacity));
64+
--spectrum-breadcrumbs-item-dragged-background: rgb(var(--spectrum-drop-zone-background-color-rgb), var(--spectrum-drop-zone-background-color-opacity));
6565

6666
/* Sub-component: heading */
6767
--mod-heading-margin-end: 0px;

components/calendar/index.css

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,7 @@
1212
*/
1313

1414
.spectrum-Calendar {
15-
--spectrum-calendar-day-background-color-selected-disabled: rgba(var(--spectrum-gray-100-rgb), 0.4);
15+
--spectrum-calendar-day-background-color-selected-disabled: rgb(var(--spectrum-gray-100-rgb), 0.4);
1616

1717
--spectrum-calendar-day-width: var(--mod-calendar-day-width, var(--spectrum-component-height-100));
1818
--spectrum-calendar-day-height: var(--mod-calendar-day-height, var(--spectrum-component-height-100));
@@ -48,7 +48,7 @@
4848
--spectrum-calendar-day-background: var(--mod-calendar-day-background-color, transparent);
4949
--spectrum-calendar-day-background-selected: var(--highcontrast-calendar-day-background-selected, var(--mod-calendar-day-background-color-selected, var(--spectrum-calendar-day-background-color-selected)));
5050
--spectrum-calendar-day-background-selected-hover: var(--highcontrast-calendar-day-background-selected-hover, var(--mod-calendar-day-background-color-selected-hover, var(--spectrum-calendar-day-background-color-selected-hover)));
51-
--spectrum-calendar-day-background-color-selected-disabled: rgba(var(--spectrum-gray-100-rgb), 0.4);
51+
--spectrum-calendar-day-background-color-selected-disabled: rgb(var(--spectrum-gray-100-rgb), 0.4);
5252
--spectrum-calendar-day-background-cap-selected: var(--highcontrast-calendar-day-background-cap-selected, var(--mod-calendar-day-background-color-cap-selected, var(--spectrum-calendar-day-background-color-cap-selected)));
5353
--spectrum-calendar-day-background-hover: var(--highcontrast-calendar-day-background-hover, var(--mod-calendar-day-background-color-hover, var(--spectrum-calendar-day-background-color-hover)));
5454
--spectrum-calendar-day-background-focus: var(--highcontrast-calendar-day-background-focus, var(--mod-calendar-day-background-color-key-focus, var(--spectrum-calendar-day-background-color-key-focus)));

components/card/dist/metadata.json

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -99,7 +99,6 @@
9999
"--mod-card-minimum-width",
100100
"--mod-card-preview-background-color",
101101
"--mod-card-preview-background-color-hover",
102-
"--mod-card-preview-border-width-selected",
103102
"--mod-card-preview-minimum-height",
104103
"--mod-card-selected-background-color-rgb",
105104
"--mod-card-selected-background-opacity",
@@ -146,7 +145,6 @@
146145
"--spectrum-card-minimum-width",
147146
"--spectrum-card-preview-background-color",
148147
"--spectrum-card-preview-background-color-hover",
149-
"--spectrum-card-preview-border-width-selected",
150148
"--spectrum-card-preview-minimum-height",
151149
"--spectrum-card-selected-background-color-rgb",
152150
"--spectrum-card-selected-background-opacity",

components/card/index.css

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -65,8 +65,6 @@
6565
/* Selected */
6666
--spectrum-card-selected-background-opacity: var(--mod-card-selected-background-opacity, 0.1); /* table-selected-row-background-opacity does not exist in tokens yet */
6767

68-
--spectrum-card-preview-border-width-selected: var(--mod-card-preview-border-width-selected, var(--spectrum-border-width-100));
69-
7068
/* Horizontal */
7169
--spectrum-card-horizontal-body-padding: var(--mod-card-horizontal-body-padding, var(--spectrum-spacing-300));
7270
--spectrum-card-horizontal-preview-padding: var(--mod-card-horizontal-preview-padding, var(--spectrum-spacing-200));
@@ -378,7 +376,7 @@
378376
}
379377

380378
&.is-selected .spectrum-Card-preview::before {
381-
background-color: rgba(var(--mod-card-selected-background-color-rgb, var(--spectrum-card-selected-background-color-rgb)), var(--mod-card-selected-background-opacity, var(--spectrum-card-selected-background-opacity)));
379+
background-color: rgb(var(--mod-card-selected-background-color-rgb, var(--spectrum-card-selected-background-color-rgb)), var(--mod-card-selected-background-opacity, var(--spectrum-card-selected-background-opacity)));
382380
}
383381

384382
.spectrum-Card-body {

components/coachmark/dist/metadata.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -103,7 +103,7 @@
103103
"--spectrum-heading-color",
104104
"--spectrum-line-height-200",
105105
"--spectrum-medium-font-weight",
106-
"--spectrum-popover-content-area-spacing",
106+
"--spectrum-popover-edge-to-content-area",
107107
"--spectrum-sans-serif-font",
108108
"--spectrum-spacing-100",
109109
"--spectrum-spacing-200",

components/coachmark/index.css

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -119,7 +119,7 @@
119119

120120
/* Wrapper element around the nested action menu sub-component */
121121
.spectrum-CoachMark-action-menu {
122-
--mod-popover-content-area-spacing: var(--spectrum-popover-content-area-spacing);
122+
--mod-popover-content-area-spacing: var(--spectrum-popover-edge-to-content-area);
123123

124124
white-space: nowrap;
125125
z-index: 1;

0 commit comments

Comments
 (0)