Skip to content

Commit eda9897

Browse files
authored
feat(thumbnail-opacity-checkboard): S2 migration (#3394)
* feat(thumbnail-opacity-checkboard): s2 migration * feat(thumbnail-opacity-checkboard): fix token rule * feat(thumbnail-opacity-checkboard): fix typo, update changeset, move rule * feat(thumbnail-opacity-checkboard): drop unnecessary styles * feat(thumbnail-opacity-checkboard): update token * feat(thumbnail-opacity-checkboard): update changeset
1 parent 8aa61a9 commit eda9897

File tree

4 files changed

+20
-44
lines changed

4 files changed

+20
-44
lines changed

.changeset/twenty-tables-know.md

+10
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,10 @@
1+
---
2+
"@spectrum-css/opacitycheckerboard": major
3+
"@spectrum-css/thumbnail": minor
4+
---
5+
6+
# thumbnail-opacity-checkerboard S2 migration
7+
8+
Adds thumbnail specific `thumbnail-opacity-checkerboard-square-size` token aliased to `--spectrum-opacity-checkerboard-size`. This impacts the display of the `opacity-checkerboard` when used within the `thumbnail` component.
9+
10+
This also corrects an issue with a token name in the `thumbnail` component by renaming `--spectrum-thumbnail-border-color-opacity` to `--spectrum-thumbnail-border-opacity`.

components/opacitycheckerboard/index.css

+9-40
Original file line numberDiff line numberDiff line change
@@ -8,53 +8,22 @@ express or implied. See the License for the specific language governing
88
permissions and limitations under the License. */
99

1010
.spectrum-OpacityCheckerboard {
11-
--spectrum-opacity-checkerboard-dark: var(
12-
--spectrum-opacity-checkerboard-square-dark
13-
);
14-
--spectrum-opacity-checkerboard-light: var(
15-
--spectrum-opacity-checkerboard-square-light
16-
);
17-
--spectrum-opacity-checkerboard-size: var(
18-
--spectrum-opacity-checkerboard-square-size-medium
19-
);
11+
--spectrum-opacity-checkerboard-dark: var(--spectrum-opacity-checkerboard-square-dark);
12+
--spectrum-opacity-checkerboard-light: var(--spectrum-opacity-checkerboard-square-light);
13+
--spectrum-opacity-checkerboard-size: var(--spectrum-opacity-checkerboard-square-size-medium);
2014
--spectrum-opacity-checkerboard-position: left top;
2115
}
2216

2317
.spectrum-OpacityCheckerboard--sizeS {
24-
--spectrum-opacity-checkerboard-size: var(
25-
--spectrum-opacity-checkerboard-square-size-small
26-
);
18+
--spectrum-opacity-checkerboard-size: var(--spectrum-opacity-checkerboard-square-size-small);
2719
}
2820

2921
.spectrum-OpacityCheckerboard {
30-
background: repeating-conic-gradient(
31-
var(
32-
--mod-opacity-checkerboard-light,
33-
var(--spectrum-opacity-checkerboard-light)
34-
)
35-
0% 25%,
36-
var(
37-
--mod-opacity-checkerboard-dark,
38-
var(--spectrum-opacity-checkerboard-dark)
39-
)
40-
0% 50%
41-
)
42-
var(
43-
--mod-opacity-checkerboard-position,
44-
var(--spectrum-opacity-checkerboard-position)
45-
) /
46-
calc(
47-
var(
48-
--mod-opacity-checkerboard-size,
49-
var(--spectrum-opacity-checkerboard-size)
50-
) * 2
51-
)
52-
calc(
53-
var(
54-
--mod-opacity-checkerboard-size,
55-
var(--spectrum-opacity-checkerboard-size)
56-
) * 2
57-
);
22+
background: repeating-conic-gradient(var(--mod-opacity-checkerboard-light, var(--spectrum-opacity-checkerboard-light)) 0% 25%, var(--mod-opacity-checkerboard-dark, var(--spectrum-opacity-checkerboard-dark)) 0% 50%) var(--mod-opacity-checkerboard-position, var(--spectrum-opacity-checkerboard-position)) / calc(var(--mod-opacity-checkerboard-size, var(--spectrum-opacity-checkerboard-size)) * 2) calc(var(--mod-opacity-checkerboard-size, var(--spectrum-opacity-checkerboard-size)) * 2);
23+
24+
.spectrum-Thumbnail & {
25+
--spectrum-opacity-checkerboard-size: var(--spectrum-thumbnail-opacity-checkerboard-square-size);
26+
}
5827
}
5928

6029
@media (forced-colors: active) {

components/thumbnail/index.css

+1-1
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,7 @@ governing permissions and limitations under the License.
1717
--spectrum-thumbnail-border-width: var(--spectrum-border-width-100);
1818

1919
/* @todo Refactor with --spectrum-thumbnail-border-color once gray rgb token is no longer necessary to workaround nested rgb color token value using rgba(). */
20-
--spectrum-thumbnail-border-color-rgba: rgba(var(--spectrum-gray-800-rgb), var(--spectrum-thumbnail-border-color-opacity));
20+
--spectrum-thumbnail-border-color-rgba: rgba(var(--spectrum-gray-800-rgb), var(--spectrum-thumbnail-border-opacity));
2121
--spectrum-thumbnail-layer-border-width-inner: var(--spectrum-border-width-400);
2222
--spectrum-thumbnail-layer-border-color-inner: var(--spectrum-white);
2323
--spectrum-thumbnail-layer-border-width-outer: var(--spectrum-border-width-100);

components/thumbnail/stories/template.js

-3
Original file line numberDiff line numberDiff line change
@@ -113,9 +113,6 @@ export const Template = ({
113113
${OpacityCheckerboard({
114114
rootClass: backgroundColor ? `${rootClass}-image-wrapper` : undefined,
115115
customClasses: isLayer ? [`${rootClass}-layer-inner`] : !backgroundColor ? [`${rootClass}-image-wrapper`] : [],
116-
customStyles: {
117-
"--spectrum-opacity-checkerboard-size": "var(--spectrum-thumbnail-opacity-checkerboard-square-size)"
118-
},
119116
content: image ? [image] : [],
120117
})}
121118
</div>

0 commit comments

Comments
 (0)