atom/github

Click Staged/Unstaged Changes header to select all files

Open

#1 800 ouverte le 15 nov. 2018

Voir sur GitHub
 (2 commentaires) (1 réaction) (0 assignés)JavaScript (1 069 stars) (469 forks)batch import
good first issuepapercut

Description

In at least couple of user interviews I participated in, the user tried to select all files by clicking the header. This behavior would be consistent with what we have in the ChangedFileItem where clicking a hunk header highlights all the changed lines in the hunk

Guide contributeur

Stack technique
javascript
Domaine
frontenddeveloper experience
Type d'issue
feature
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.
fresh
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
clear
Prérequis
Familiarity with Atom package developmentBasic knowledge of Git staging UI
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
60
Direction de recherche
Examine the Click handling in the ChangedFileItem component (likely in packages/github/lib/views/ changed file item.js) to see how hunk header clicks select all lines. Then apply similar logic to the Staged/Unstaged Changes header components (e.g., staged changes view and unstaged changes view). The issue mentions that users expect clicking the header to select all files, so look at the header's click event and implement a handler that checks all file checkboxes within that section.