DiamondLightSource / DiamondLightSource/sci-react-ui

Investigate transition suppression for nested `data-mode`

Ouverte
#258 0 commentaires 0 réactions 0 personnes assignées Voir sur GitHub
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

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.