DiamondLightSource / DiamondLightSource/sci-react-ui

NavLinks, FooterLinks, NavLink and FooterLink should be using `styled`.

Ouverte
#89 0 commentaires 0 réactions 0 personnes assignées Voir sur GitHub
accepted priority: low
Langage dominant
TypeScript
Étoiles
8
Forks
3
Merge moyen
3 j 15 h
PR mergées (30 j)
5

Description

Currently NavLinks, FooterLinks, NavLink and FooterLink styles can all be removed by passing in `styles={}` , obviously, breaking the way they are displayed.

Other components are using the function `styled()`, e.g.
```
const BarStyled = styled(Bar)(
({ theme }) => ({
backgroundColor: theme.vars.palette.primary.light
})
);
```

Guide de contribution

Ouvrir le guide de contribution

Piste de recherche

Repérez les composants NavLinks, FooterLinks, NavLink et FooterLink et comparez leur style à celui d’autres composants qui utilisent styled(). Vérifiez que le fait de passer styles={} ne peut plus supprimer les styles nécessaires à leur affichage, puis vérifiez le comportement des composants concernés.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Évaluation

Stack technique
react, typescript
Domaine
frontend
Type d'issue
Refactorisation
Difficulté
3/5
Temps estimé
1-2 jours
Activité
À l'abandon
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.