Skip to content
This repository has been archived by the owner on Jan 10, 2022. It is now read-only.

Supprimer l'utilisation de Vuetify #55

Closed
Haelle opened this issue Jul 16, 2019 · 8 comments
Closed

Supprimer l'utilisation de Vuetify #55

Haelle opened this issue Jul 16, 2019 · 8 comments
Assignees

Comments

@Haelle
Copy link
Contributor

Haelle commented Jul 16, 2019

Supprimer toute dépendance à Vuetify.

Cf PR #43 où Vuetify où a été ajouté.
Cf etalab/template.data.gouv.fr#121 où le tableau a été ajouté au template

En gros revoir le code de ce fichier pour utiliser le design du template
src/components/resource/user/Index.vue

@Samuelfaure
Copy link
Contributor

Note : supprimer vuetify = devoir recoder la recherche et le ordering du tableau.

@Samuelfaure Samuelfaure added wontfix This will not be worked on and removed wontfix This will not be worked on labels Jul 23, 2019
@brindu
Copy link
Contributor

brindu commented Jul 23, 2019

Je ne pense pas que ça prenne trop de temps de devoir recoder le JS pour les filtres et tris du tableau, c'est moins quickwin que de juste utiliser le template CSS mais les fonctionnalités sont assez simples à mettre en place.

@Samuelfaure
Copy link
Contributor

Le js pour le filtre m'a pris 5 mins mais le ordering c'est davantage compliqué, il faudra un peu de temps pour faire ça propre sinon on va bloat le component. Mais oui ce sera a faire probablement.

@brindu
Copy link
Contributor

brindu commented Jul 23, 2019

Bloat le component ?

Un simple getter ordonné ne fait pas le taf ici ?

@Samuelfaure
Copy link
Contributor

Faut gérer le click sur les fleches, qui :

  • ordonne le tableau dans le bon sens selon quelle flèche est cliquée
  • s'intègre bien avec le filtrage du tableau par la fonction recherche
  • change sa propre classe en descending et la classe des autres flèches en ascending (ou le contraire)

pas que ce soit très difficile mais c'est pas non plus 5 mins pour faire ça bien et propre

@Haelle
Copy link
Contributor Author

Haelle commented Jul 24, 2019

Il faut aussi penser à la recherche sur un champ invisible (le uuid de l'utilisateur), je m'en sers tout le temps pour identifier les utilisateurs depuis Kibana

@Samuelfaure
Copy link
Contributor

Samuelfaure commented Jul 24, 2019

@Haelle Ofc, ceci ne rajoute pas de complexité en fait. Il suffit de faire :

function arrayFiltered () {
  if (this.search == '') { return this.array}

  search = new RegExp(this.search, "gi")

  this.array.filter((item) => {
    return( item.uuid.match(search) || item.context.match(search) || item.email.match(search))
  })
}

Pas très beau mais ça fait le job

@brindu
Copy link
Contributor

brindu commented Oct 10, 2019

Closed in #75

@brindu brindu closed this as completed Oct 10, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants