DiamondLightSource / DiamondLightSource/sci-react-ui

Investigate transition suppression for nested `data-mode`

Offen
#258 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen
Vorherrschende Sprache
TypeScript
Sterne
8
Forks
3
Ø Merge
3 T. 15 Std.
Gemergte PRs (30 T.)
5

Beschreibung

Based on changes in #220, `data-mode` could be applied to any container, not just the document root. However, the `ThemeProvider` transition suppression currently only watches `document.documentElement`.

This works for our current Storybook use case, where nested themes are static, but it may not be sufficient if we later support components that switch between light and dark mode independently (for example, code editors or previews).

**Task**

- Review whether transition suppression should also support nested `data-mode` containers.
- Decide whether the current behaviour is sufficient or should be extended.

Beitragsleitfaden

Beitragsleitfaden öffnen

Rechercherichtung

Beginne mit der Überprüfung des Verhaltens von ThemeProvider zur Unterdrückung von Übergängen und der Änderungen in Issue #220, wobei der Schwerpunkt darauf liegt, wie document.documentElement derzeit beobachtet wird. Vergleiche dieses Verhalten mit verschachtelten data-mode-Containern und dokumentiere anschließend, ob das aktuelle Verhalten ausreichend ist, oder definiere den erforderlichen Umfang, um das unabhängige Umschalten verschachtelter Themes zu unterstützen.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
typescript
Bereich
frontend
Issue-Typ
Feature
Schwierigkeit
4/5
Geschätzter Aufwand
3-5 Tage
Aktivitätsstatus
Ruhig
Klarheit
Größtenteils klar
Anfängerfreundlichkeit
45/100

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.