Triage alphabétique vertical des colonnes avec des checkboxes dans l'onglet produit #1470
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
pletelli
reviewed
Jan 16, 2025
| /* | ||
| Retourne le breakpoint actif depuis notre config tailwind. À noter que | ||
| les breakpoints doivent être triés dans le fichier de configuration. | ||
| */ |
There was a problem hiding this comment.
Ici, ajouter le même commentaire sur les breakpoints qui doivent être triés dans le fichier tailwind.config.js ?
Ou bien modifier la function pour qu'elle ne retourne la valeur min qu'une fois toutes les valeurs parcourues ?
There was a problem hiding this comment.
Oui en vrai plus j'y pense plus la fonction devrait faire ce triage par sécurité. Surtout que l'ordre des clés dans un objet JS n'est pas garanti. Je fais ce changement.
There was a problem hiding this comment.
Fix ici : f6820d6 ;)
pletelli
approved these changes
Jan 16, 2025
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Lié à #994
Contexte
L'ordre alphabétique des checkboxes a été rémonté comme problème plusieurs fois. Aujourd'hui la lecture est horizontale alors que les personnes s'attendent à l'avoir en vertical. Capture d'écran aujourd'hui :
Scope
Le placement d'éléments côté HTML se fait de façon horizontale, donc on doit manipuler le tableau pour que le placement change.
Par exemple, le tableau trié
[0, 1, 2, 3, 4, 5, 6, 7, 8]en trois colonnes s'afficherait :Or, il faudrait transformer ce tableau à
[0, 3, 6, 1, 4, 7, 2, 5, 8]pour pouvoir l'afficher en lecture verticale :La fonction
transformArrayByColumnse charge de ceci.De plus, il faut savoir sur quel breakpoint on se trouve pour connaître le nombre de colonnes et donc transformer correctement le tableau. La fonction
getCurrentBreakpointse charge de cela.En alimentant cette fonction dans le watcher pour la dimension de la fenêtre, on affiche le bon ordre même lorsque la fenêtre est redimensionnée.
Démo