Skip to content

Commit 7d3047a

Browse files
authored
fix some bugs and add missing comment
1 parent b81ff43 commit 7d3047a

File tree

2 files changed

+24
-0
lines changed

2 files changed

+24
-0
lines changed

packages/ui-components/styles/index.css

Lines changed: 20 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -12,3 +12,23 @@
1212
@import './effects.css';
1313
@import './base.css' layer(utilities);
1414
@import './markdown.css' layer(utilities);
15+
16+
/*
17+
The default border color has changed to `currentColor` in Tailwind CSS v4,
18+
so we've added these compatibility styles to make sure everything still
19+
looks the same as it did with Tailwind CSS v3.
20+
21+
If we ever want to remove these styles, we need to add an explicit border
22+
color utility to any element that depends on these defaults.
23+
*/
24+
@layer base {
25+
*,
26+
::after,
27+
::before,
28+
::backdrop,
29+
::file-selector-button {
30+
border-color: var(--color-gray-200, currentColor);
31+
}
32+
}
33+
34+
@custom-variant aria-current (&[aria-current="page"]);

packages/ui-components/styles/theme.css

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -81,6 +81,10 @@
8181
--shadow-lg:
8282
0px 4px 6px -2px var(--color-shadow, rgba(0, 0, 0 / 3%)),
8383
0px 12px 16px -4px var(--color-shadow, rgba(0, 0, 0 / 8%));
84+
85+
/* TODO(@avivkeller): The --font-xyz-xyz variable is assigned by Next.js in apps/site.
86+
Ideally, this package shouldn't depend on the consuming application's
87+
variables, it should remain self-contained. */
8488
--font-open-sans: var(--font-open-sans);
8589
--font-ibm-plex-mono: var(--font-ibm-plex-mono);
8690
--font-weight-regular: 400;

0 commit comments

Comments
 (0)