Projet

Général

Profil

Development #11491

étendre la vue de gestions des ressources

Ajouté par Frédéric Péters il y a presque 8 ans. Mis à jour il y a plus de 5 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
-
Version cible:
-
Début:
22 juin 2016
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:

Description

La vue /manage/assets est pour le moment des plus basiques, elle devrait gagner en fonctionnalités : permettre de supprimer des fichiers (en vérifiant s'ils ne sont pas utilisés dans la plateforme), permettre d'écraser un fichier par un nouveau, s'afficher en vue liste ou en vue icône, avoir une pagination, etc.


Fichiers

0001-Changes-for-ticket-11491.patch (13,2 ko) 0001-Changes-for-ticket-11491.patch Jean-Baptiste Jaillet, 29 juin 2016 17:14
0001-manager-change-file-handling-11491.patch (2,14 ko) 0001-manager-change-file-handling-11491.patch Jean-Baptiste Jaillet, 29 juin 2016 17:15
0002-manager-Add-pagination-for-assets-11491.patch (2,26 ko) 0002-manager-Add-pagination-for-assets-11491.patch Jean-Baptiste Jaillet, 29 juin 2016 17:15
0001-manager-new-handle-for-assets-11491.patch (14,8 ko) 0001-manager-new-handle-for-assets-11491.patch Jean-Baptiste Jaillet, 19 juillet 2016 10:18
0001-manager-new-handle-for-assets-11491.patch (14,9 ko) 0001-manager-new-handle-for-assets-11491.patch Jean-Baptiste Jaillet, 17 février 2017 14:39
0003-manager-add-upload-from-assets-management-page-11491.patch (5,48 ko) 0003-manager-add-upload-from-assets-management-page-11491.patch Frédéric Péters, 25 décembre 2017 19:52
0004-manager-add-action-to-overwrite-an-asset-11491.patch (5,89 ko) 0004-manager-add-action-to-overwrite-an-asset-11491.patch Frédéric Péters, 25 décembre 2017 19:52
0002-manager-add-minimal-search-of-assets-11491.patch (3,29 ko) 0002-manager-add-minimal-search-of-assets-11491.patch Frédéric Péters, 25 décembre 2017 19:52
0001-manager-revamp-assets-management-page-11491.patch (11,2 ko) 0001-manager-revamp-assets-management-page-11491.patch Frédéric Péters, 25 décembre 2017 19:52
Screenshot-2017-12-25 Portail.png (133 ko) Screenshot-2017-12-25 Portail.png Frédéric Péters, 25 décembre 2017 20:08
Capture d’écran de 2017-12-29 19-27-03.png (15,2 ko) Capture d’écran de 2017-12-29 19-27-03.png Thomas Noël, 29 décembre 2017 19:28
0001-manager-revamp-assets-management-page-11491.patch (12,2 ko) 0001-manager-revamp-assets-management-page-11491.patch Frédéric Péters, 30 décembre 2017 11:25

Demandes liées

Lié à Combo - Development #9592: Ajout d'un bloc de contenu de type documentInformation nécessaire11 janvier 2016

Actions
Lié à Combo - Development #21008: Améliorations possibles sur interface "Ressources"Rejeté05 janvier 2018

Actions

Révisions associées

Révision be2a10e4 (diff)
Ajouté par Frédéric Péters il y a plus de 6 ans

manager: revamp assets management page (#11491)

Révision a652a983 (diff)
Ajouté par Frédéric Péters il y a plus de 6 ans

manager: add minimal search of assets (#11491)

Révision 8f6e1027 (diff)
Ajouté par Frédéric Péters il y a plus de 6 ans

manager: add upload from assets management page (#11491)

Révision f23083b9 (diff)
Ajouté par Frédéric Péters il y a plus de 6 ans

manager: add action to overwrite an asset (#11491)

Historique

#1

Mis à jour par Jean-Baptiste Jaillet il y a presque 8 ans

  • Fichier 0001-Changes-for-ticket-11491.patch ajouté
  • Statut changé de Nouveau à En cours

Patch ajouté pour la gestion des icônes des fichiers qui ne sont pas des images, ajout d'icônes pour changer l'affichage et gestion de remplacement de fichiers existants.
Ça fait un gros patch, j'ai splitté là du coup, il ne me reste que la pagination.

#2

Mis à jour par Frédéric Péters il y a presque 8 ans

Même commentaire que dans #11175 pour le formatage git (email, message).

Sur le patch en lui-même, plutôt qu'ajouter des classes "left" partout sur un clic sur le mode "thumbnails", je ferais juste un toggleClass sur le div#assets-thumbs, et je gérerais les modifs de présentation via la CSS.

À propos des CSS aussi, on essaie d'avoir des classes qui correspondent plutôt à des rôles et non à un rendu particulier, par exemple, ".mg-left-10", ça ne le fait pas. On ferait plutôt là #assets-cmds button { margin-left: 10px; }.

Pour les deux icônes de bascule, ça pourrait être textuellement dans la page "Mosaïque" et "Liste" et la substitution pour des icônes réalisée dans le fichier css. Voire même, en fait, comme on a la place, on pourrait laisser le texte présent dans la page : <span id="...">Vue en mosaïque</span> <span id="...">Vue en liste</span> et dans la CSS #list::before { content: "\f03a;"; font-family: FontAwesome; }. (et ça pourrait peut-être même être des <button> plutôt que des <span> ou des <i>).

Sinon en détail il y a des espaces en fin de ligne à un endroit (dans asset_file_replace, une ligne vide qui en fait contient huit espaces), à supprimer. (il y a éventuellement une config d'éditeur pour les mettre en évidence).

#3

Mis à jour par Jean-Baptiste Jaillet il y a presque 8 ans

  • Fichier 0001-Changes-for-ticket-11491.patch supprimé
#4

Mis à jour par Jean-Baptiste Jaillet il y a presque 8 ans

Ajout de remplacement de fichiers identiques (pdf / pdf, doc / doc ...) sans changer l'url (comme pour les images) afin de préserver les liens.
Ajout de la pagination pour la page /manage/assets.

Je réupload l'autre patch avec la bonne adresse mail

#5

Mis à jour par Frédéric Péters il y a presque 8 ans

  • Patch proposed changé de Non à Oui

Tu peux réunir le tout dans un patch unique, avec un message de commit adéquat ?.

Sur le patch en lui-même, plutôt qu'ajouter des classes "left" partout sur un clic sur le mode "thumbnails", je ferais juste un toggleClass sur le div#assets-thumbs, et je gérerais les modifs de présentation via la CSS.

Pour les deux icônes de bascule, ça pourrait être textuellement dans la page "Mosaïque" et "Liste" et la substitution pour des icônes réalisée dans le fichier css. Voire même, en fait, comme on a la place, on pourrait laisser le texte présent dans la page : <span id="...">Vue en mosaïque</span> <span id="...">Vue en liste</span> et dans la CSS #list::before { content: "\f03a;"; font-family: FontAwesome; }. (et ça pourrait peut-être même être des <button> plutôt que des <span> ou des <i>).

Cela reste pour moi d'actualité; tu pourras regarder ?

#6

Mis à jour par Jean-Baptiste Jaillet il y a presque 8 ans

Bon, j'ai fait un grand mic mac avec git pour tout remettre d'aplomb^^
J'ai fait les modifications qu'il restait encore.

#7

Mis à jour par Frédéric Péters il y a environ 7 ans

#8

Mis à jour par Frédéric Péters il y a environ 7 ans

Après ce temps, JB, ça te dirait de relire ton patch, voir pour toi-même ce qu'il lui faudrait encore et reprendre ce travail ?

#9

Mis à jour par Jean-Baptiste Jaillet il y a environ 7 ans

Hum, j'ai relu il me semble que les dernières remarques ont été implémentées et ça marche chez moi en local.
Je veux bien une relecture s'il y a des points qui peuvent être changés encore. J'ai rebasé le commit sur le master.

#10

Mis à jour par Frédéric Péters il y a environ 7 ans

Tout bonnement, facile à voir d'un œil : le message de commit est un mélange horrible, il n'y a pas de tests.

Après, à lire le code, il y a un tas de détails sur lesquels tu aurais pu tiquer maintenant, parce que mentionnés dans des relectures sur d'autres tickets, genre content: '\f03a'; c'est important d'ajouter en commentaire derrière le nom de l'icône FontAwesome. Ou simplement parce qu'à relire ton patch tu pourrais te demander mais pourquoi donc avoir mis une classe mg-lft-10 qui ne sert apparemment à rien. Ou vu récemment dans gadjo que le fade out automatique des notifications, on ne fait plus. Etc.

Bref, voici déjà cinq points à améliorer, mais je t'invite d'abord à relire ton patch et à noter cinq autres points, avant d'attacher un nouveau patch.

#11

Mis à jour par Frédéric Péters il y a plus de 6 ans

  • Assigné à Jean-Baptiste Jaillet supprimé
#12

Mis à jour par Frédéric Péters il y a plus de 6 ans

  • Patch proposed changé de Oui à Non
#13

Mis à jour par Frédéric Péters il y a plus de 6 ans

Reprise à zéro :

  • 0001 : un peu gros mais les modifications s'entremêlaient :
    • transformation de la vue en ListView pour gagner la pagination (même si ça fonctionne ici sans modèle je trouve ça plus clean que de gérer Paginator & friends manuellement)
    • modification du rendu pour être dans un tableau (icône selon le type de fichier / nom de fichier / taille / vignette / actions)
    • transformation de la suppression en DeleteView pour avoir demande de confirmation
  • 0002 : ajout d'un champ de recherche, simplement basé sur le nom de fichier mais déjà ça peut être utile, et ça mime la page des transactions
  • 0003 : ajout d'une action "upload" directement depuis cette page; le gros intérêt ici c'est simplifier le dépôt d'un fichier (genre pdf) dans combo, pour exploitation peu importe où. (parce que passer par créer une page, y mettre une cellule texte, faire un lien, aller dans le troisième onglet de la boite de dialogue, etc. ce n'est pas terrible).
  • 0004 : ajout d'une action permettant de remplacer un fichier existant; ici aussi c'est en lien avec l'utilisation "dépôt de fichier" : après avoir posé un pdf, s'il faut le mettre à jour, c'est totalement galère de devoir aller modifier tous les endroits où il était référencé pour y mettre une nouvelle URL.

Des trucs que j'ai noté, qui ne sont pas dans cette série, qui pourraient venir après, dans d'autres tickets :

  • la possibilité de voir où une ressource est utilisée (dans la limite de combo)
  • l'import/export des ressources (mais plutôt qu'un truc spécifique ça doit à mon avis intégréer l'import/export de site, général)
  • l'amélioration des vues "explorer" de ckeditor en les remplaçant par une variante de celle-ci
  • l'ajout d'une action "afficher le lien" qui pour une ressource en afficherait le lien dans une zone prête à copier/coller (utile pour les fichiers qui se téléchargent, ou il n'y a pas moyen de copier depuis la barre d'adresse, ou il faut passer par clic droit / copier l'adresse du lien)
#15

Mis à jour par Frédéric Péters il y a plus de 6 ans

Et parce que la série de patchs s'appliquait en local derrière d'autres patchs, je viens de les pousser dans une branche wip/assets-management, https://git.entrouvert.org/combo.git/log/?h=wip/assets-management

#16

Mis à jour par Thomas Noël il y a plus de 6 ans

  • Fichier Capture d’écran de 2017-12-29 19-25-56.png ajouté

Petit soucis avec un odt qui n'aura pas de ::before dans la CSS : le tableau se retrouve un peu moche, cf image attachée.

#17

Mis à jour par Thomas Noël il y a plus de 6 ans

  • Fichier Capture d’écran de 2017-12-29 19-25-56.png supprimé
#20

Mis à jour par Thomas Noël il y a plus de 6 ans

Y'a un truc qui a sauté : à la droite du tableau n'apparait plus le preview de l'objet (asset-preview) ; qui ré-apparait quand même au moment où on clique sur "supprimer".

#21

Mis à jour par Frédéric Péters il y a plus de 6 ans

J'avais oublié que j'avais un travail plus récent dans une branche dédiée, était reparti de ce qui trainait encore dans mon master.

La branche http://git.entrouvert.org/combo.git/log/?h=wip/assets-management est désormais à jour (et correcte).

#22

Mis à jour par Thomas Noël il y a plus de 6 ans

Et ça m'a l'air tout bon (code comme résultat à l'écran).

#23

Mis à jour par Frédéric Péters il y a plus de 6 ans

  • Statut changé de En cours à Résolu (à déployer)
commit f23083b9b68675cf8e414fe2e400241509e67803
Author: Frédéric Péters <fpeters@entrouvert.com>
Date:   Mon Dec 25 19:08:26 2017 +0100

    manager: add action to overwrite an asset (#11491)

commit 8f6e102720721c5f3cda03238f2dcf6f74a723cf
Author: Frédéric Péters <fpeters@entrouvert.com>
Date:   Mon Dec 25 18:27:34 2017 +0100

    manager: add upload from assets management page (#11491)

commit a652a983b69686f7a1a1cf5e855d271611090dc0
Author: Frédéric Péters <fpeters@entrouvert.com>
Date:   Mon Dec 25 18:05:50 2017 +0100

    manager: add minimal search of assets (#11491)

commit be2a10e460b954d12d721f8df0cfee59ea80cc54
Author: Frédéric Péters <fpeters@entrouvert.com>
Date:   Sun Dec 24 16:58:10 2017 +0100

    manager: revamp assets management page (#11491)
#24

Mis à jour par Brice Mallet il y a plus de 6 ans

#25

Mis à jour par Frédéric Péters il y a plus de 5 ans

  • Statut changé de Résolu (à déployer) à Solution déployée

Formats disponibles : Atom PDF