DiamondLightSource / DiamondLightSource/sci-react-ui
Investigate transition suppression for nested `data-mode`
- 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
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