DiamondLightSource / DiamondLightSource/sci-react-ui
Investigate transition suppression for nested `data-mode`
- Langage dominant
- TypeScript
- Étoiles
- 8
- Forks
- 3
- Merge moyen
- 3 j 15 h
- PR mergées (30 j)
- 5
Description
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.
Guide de contribution
Ouvrir le guide de contribution
Piste de recherche
Commencez par examiner le comportement de suppression des transitions de ThemeProvider et les modifications de l’issue #220, en vous concentrant sur la manière dont document.documentElement est observé actuellement. Comparez ce comportement à celui de conteneurs data-mode imbriqués, puis documentez si le comportement actuel est suffisant ou définissez la portée nécessaire pour prendre en charge le changement indépendant de thèmes imbriqués.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Évaluation
- Stack technique
- typescript
- Domaine
- frontend
- Type d'issue
- Fonctionnalité
- Difficulté
- 4/5
- Temps estimé
- 3-5 jours
- Activité
- Calme
- Clarté
- Plutôt claire
- Accessibilité débutants
- 45/100