mattermost-community/focalboard

Revisit and fix todos in TableHeaderMenu.tsx.

Ouverte

#1 717 ouverte le 2 nov. 2021

Voir sur GitHubGuide contributeur
 (0 commentaire) (0 réaction) (0 personne assignée)TypeScript (2 542 forks)user submission
Help WantedUp for grabs

Métriques du dépôt

Stars
 (26 187 étoiles)
Métriques de merge PR
 (Aucune PR mergée en 30 j)

Description

Investigate and determine if the TODOs should still be done or removed from the code.

These deal with inserting additional properties (columns) while on the title column. Currently, we just fail silently.

Insert Right should be able to be implemented without issue, inserting an item to the right of the title column should not be an issue. https://github.com/mattermost/focalboard/blob/main/webapp/src/components/table/tableHeaderMenu.tsx#L54

Insert Left, we should determine if this functionality is still necessary. If not, remove item from menu when on the title column. https://github.com/mattermost/focalboard/blob/main/webapp/src/components/table/tableHeaderMenu.tsx#L42

Guide contributeur

Direction de recherche
Examinez les commentaires TODO dans TableHeaderMenu.tsx, en particulier aux lignes 42 et 54. Déterminez si la fonctionnalité d'insertion à gauche est toujours nécessaire ; si ce n'est pas le cas, supprimez l'élément de menu. Pour l'insertion à droite, implémentez la fonctionnalité permettant d'ajouter une colonne à droite de la colonne de titre.
Stack technique
typescriptreact
Domaine
frontend
Type d'issue
Refactorisation
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.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Récente
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
GitReactTypeScript
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
75