Add title to preset themes
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 3/5
- Temps estimé
- 1-2 jours
- Accessibilité débutants
- 38/100
- Type d'issue
- Bug
- Clarté
- Plutôt claire
- Activité
- À l'abandon
- Stack technique
- javascript
- Domaine
- accessibility, frontend
Piste de recherche
Commencez par examiner src/web/lib/components/PresetThemeSelector/index.js à la ligne indiquée et src/lib/themes.js autour de normalizeTheme(). Vérifiez comment les titres sont fournis aux thèmes prédéfinis et enregistrés, puis vérifiez le comportement au survol sur le site github.io ou en local. La tâche est terminée lorsque les thèmes prédéfinis exposent un titre pertinent et que le comportement des thèmes enregistrés est explicitement pris en compte.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
When hovering over a preset theme, we should probably set a title attribute with the theme's title.
In fact, it looks like we do, but it isn't showing up for me locally, or on the github.io site.
Not sure if we need to tweak normalizeTheme() to possibly account for a title...
https://github.com/mozilla/Themer/blob/413aa25b78865392779fdb680735d52d32568d03/src/lib/themes.js#L45-L67
Not sure how we could add a meaningful title to the saved themes since I don't think we have the UI currently for the user to set a custom title. And I don't how how useful a generated title would be. We could do something boring like `title="Saved theme 3", but not sure how useful that'd be for a11y or screen readers. Especially since I think the saved themes get saved to the front of the queue so your numbering would always change by 1 when you add a new theme.
- Langage dominant
- JavaScript
- Étoiles
- 506
- Forks
- 100
- Métriques de merge des PR
- Aucune PR mergée en 30 j
Préparer son environnement
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de mozilla/FirefoxColor
-
Difficulté 2/5 1-3 heures Accessibilité débutants 68/100
mozilla/FirefoxColor#1081 · 1 commentaire ·
-
does not see browserOuverte
Difficulté 3/5 1-2 jours Accessibilité débutants 35/100
mozilla/FirefoxColor#1087 · 1 commentaire ·
-
Difficulté 5/5 Plus d'une semaine Accessibilité débutants 35/100
mozilla/FirefoxColor#1085 · 1 commentaire ·
-
Difficulté 3/5 1-2 jours Accessibilité débutants 68/100
mozilla/FirefoxColor#1084 · 1 commentaire ·
-
Difficulté 5/5 Plus d'une semaine Accessibilité débutants 35/100
mozilla/FirefoxColor#1083 · 1 réaction ·
Toutes les issues de mozilla/FirefoxColor
Issues similaires
-
Difficulté 2/5 1-3 heures Accessibilité débutants 68/100
-
Difficulté 2/5 1-3 heures Accessibilité débutants 74/100
solana-foundation/solana-com#2202 ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 1/5 Moins d'une heure Accessibilité débutants 92/100
yegor256/copyrights-action#216 ·
-
[BUG] onScroll never refreshes the window container's height after a vertical-only viewport resizeOuverte
Difficulté 2/5 1-3 heures Accessibilité débutants 88/100
juliangarnier/anime#1192 ·
-
Difficulté 2/5 1-3 heures Accessibilité débutants 84/100
PedestrianDynamics/pyFDS-Evac#226 ·
Les mainteneurs répondent en général sous 1 jour