iodide-project/iodide

EXPLORE/REPORT button should just be one instance of one component

Open

#1 673 ouverte le 26 mars 2019

Voir sur GitHub
 (3 commentaires) (0 réactions) (0 assignés)JavaScript (1 483 stars) (147 forks)batch import
UI appearancebugclient UXgood first issue

Description

the explore/report button is currently actually two components. it should really just be one component that toggles its text+appearance, because it's conceptually just one thing, and not moving it during the transition will also fix a tooltip bug

Guide contributeur

Stack technique
javascriptreact
Domaine
frontend
Type d'issue
refactor
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
half day
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
stale
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
clear
Prérequis
React component basicsunderstanding of button's usage
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
70
Direction de recherche
Locate the two separate components for the explore and report buttons, likely in the components directory. Consolidate them into a single component with a prop to toggle text and appearance. Ensure the tooltip bug (mentioned as caused by moving the button) is fixed by keeping the same DOM element. Review the 3 existing comments for any additional context or requirements.