[Bug]: Internal and external design token hashes don't match
Los mantenedores suelen responder en 2 días
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 4/5
- Tiempo estimado
- 3-5 días
- Aptitud para principiantes
- 45/100
- Tipo de issue
- Error
- Claridad
- Bastante claro
- Estado de actividad
- Activo
- Stack tecnológico
- react, typescript
- Área
- design
Línea de trabajo
Start by comparing how @cloudscape-design/design-tokens generates or consumes CSS variable names with how @cloudscape-design/components and @cloudscape-design/global-styles emit them; the issue does not name source files or tests. Reproduce with the listed package versions and applyMode(Mode.Dark), then check whether the token variable set by dark mode is the one used by the inline style. Done when the packages' supported version contract is clear and the documented usage produces the dark-mode background.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Browser
Chrome
Package version
3.0.1330
React version
Description
I fully admit I may not be understanding something here about how this is supposed to be used, but I'm going by (as far as I can tell) official documentation.
I'm trying to implement dark mode in my application, and I have had to override some color somewhere in one of my controls (you may already feel where this is going).
As per the design token documentation, I am doing the following:
import * as awsui from '@cloudscape-design/design-tokens';
return (
<div style={{ backgroundColor: awsui.colorBackgroundContainerContent }}></div>
);
The dark mode documentation says that I should be using design tokens everywhere, which I'm doing, so I'm feeling good.
By the way, that page has 0 context on how to actually apply dark mode itself, which I've had to Google to come out to something like this:
import { applyMode, Mode } from "@cloudscape-design/global-styles";
export type LightDarkPreference = 'system' | 'dark' | 'light';
let userPreference = localStorage.getItem('light-dark-preference') as LightDarkPreference | null ?? 'system';
const systemDarkMode = window.matchMedia('(prefers-color-scheme: dark)');
// Initialize color mode based on user's OS preference and stay in sync if the preference changes.
systemDarkMode.addEventListener('change', () => refreshDarkMode());
refreshDarkMode();
function refreshDarkMode() {
if (userPreference === 'system') {
applyMode(systemDarkMode.matches ? Mode.Dark : Mode.Light);
} else if (userPreference === 'dark') {
applyMode(Mode.Dark);
} else {
applyMode(Mode.Light);
}
}
The symptom
But of course, when I switch to Dark Mode my background is still bright white. The design token didn't do anything:
The promixate cause
When I look at the CSS it's because the variable names don't match:
// Variable consumed by the design-tokens library
background-color: var(--color-background-container-content-mjfil9, #ffffff);
// Variable set by the <body> element
.awsui-dark-mode:not(#\ ) {
--color-background-container-content-6u8rvp: var(--color-neutral-850-3f0gro);
}
The root cause?
Because mjfil9 != 6u8rvp, the variable that is set is not the variable that is read.
But my question is: why? I'm following documentation (as far as I can tell).
I'm consuming the following packages... they're not all the same major version but they are all the same major version line as the most recent packges (v3 for components and design-tokens, v1 for global-styles). I could perhaps expect incompatibilities between major version lines, but not inside a major version line.
@cloudscape-design/components 3.0.1330
@cloudscape-design/design-tokens 3.0.114
@cloudscape-design/global-styles 1.0.62
As far as I can tell I also only have one copy of each, so it's not multiple copies of the same package at slightly different versions, which should have been deduplicated but weren't.
What is the contract here?
Please tell me the it isn't "I have to go hunt for the exact patch version of components that was used to build design-tokens (or the other way around), and any time I upgrade one of the packages I have to go hunt for the patch version again" ?
Source code
No response
Reproduction
No response
Code of Conduct
- I agree to follow this project's Code of Conduct
- I checked the current issues for duplicate problems
- Lenguaje dominante
- TypeScript
- Estrellas
- 2.7k
- Forks
- 248
- Merge medio
- 3 d 14 h
- PR fusionados (30 d)
- 83
Preparar el entorno
Este proyecto no incluye contenedor de desarrollo, Dockerfile ni guía de contribución, así que la configuración corre por tu cuenta: empieza por su README y consulta nuestra guía para la primera contribución para los pasos generales.
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de cloudscape-design/components
-
[Bug]: Circular app layout navigation and help icons are shifted upward in the official demoAbiertobug
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
cloudscape-design/components#5184 · 1 comentario ·
Los mantenedores suelen responder en 2 días
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
cloudscape-design/components#5152 · 1 comentario ·
Los mantenedores suelen responder en 2 días
-
enhancement
Dificultad 5/5 Más de una semana Aptitud para principiantes 15/100
cloudscape-design/components#5149 · 1 comentario ·
Los mantenedores suelen responder en 2 días
-
Dificultad 5/5 Más de una semana Aptitud para principiantes 35/100
cloudscape-design/components#5106 · 1 comentario ·
Los mantenedores suelen responder en 2 días
-
[Bug/Feature Request]: `usePerformanceMarks` has ID generation incompatible with Next.js Partial PrerenderingPosiblemente ocupada @amanabiy la tomó hace 22 días. Abiertoenhancement
cloudscape-design/components#5023 · 1 comentario · 1 asignado ·
Los mantenedores suelen responder en 2 días
Todos los issues de cloudscape-design/components
Issues similares
-
area: image-engine bug priority: low
Dificultad 2/5 1-3 horas Aptitud para principiantes 88/100
snapotter-hq/SnapOtter#3885 ·
Los mantenedores suelen responder en 1 día
-
Mend: dependency security vulnerability
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
Los mantenedores suelen responder en 3 días
-
pending triage
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
Los mantenedores suelen responder en 1 día
-
bug P3 UI/UX
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
Los mantenedores suelen responder en 1 día
-
clawsweeper:fix-shape-clear clawsweeper:queueable-fix clawsweeper:source-repro impact:ux-friction issue-rating: 🦞 diamond lobster no-stale P3
Dificultad 2/5 1-3 horas Aptitud para principiantes 73/100
openclaw/openclaw#182077 · 1 comentario · 1 reacción ·
Los mantenedores suelen responder en 1 día