patternfly / patternfly/patternfly-react

Bug - Table - Refactor Column Management with Draggable demo

Ouverte
#7,471 1 commentaire 0 réactions 0 personnes assignées Voir sur GitHub

Personne n'a encore pris cette issue.

Pinned
Langage dominant
TypeScript
Étoiles
862
Forks
392
Merge moyen
4 j 8 h
PR mergées (30 j)
9

Description

Describe the problem
A clear and concise description of the problem. Which components are affected?

@jenny-s51 This looks great, but there seems to be a bug with column reordering via drag & drop. If I reorder columns and then click Save, upon returning to the table, it's empty.

Originally posted by @mcarrano in https://github.com/patternfly/patternfly-react/pull/7446#pullrequestreview-986229584

@mcarrano noticed this bug upon review of #7446 ... this bug already existed in the column management draggable demo. Looks like we're using DataList's deprecated onDrag API here, along with the deprecated itemOrder prop.

In order to fix this bug, we need to refactor this demo to use DragDrop with DataList here.

Guide de contribution

Ouvrir le guide de contribution

Par où commencer

  1. Lisez l'issue en entier, puis le guide de contribution du projet.
  2. Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
  3. Forkez le dépôt et travaillez sur une branche.
  4. Ouvrez une pull request qui référence le numéro de l'issue.

Piste de recherche

Commencez par la démo glisser-déposer de gestion des colonnes et examinez son utilisation de DataList, en particulier l’API onDrag obsolète et la prop itemOrder. Refactorez-la pour utiliser DragDrop avec DataList, puis vérifiez que le réordonnancement des colonnes, l’enregistrement et le retour à la table conservent les colonnes au lieu d’afficher une table vide.

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
35/100

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.