Development #11491
étendre la vue de gestions des ressources
0%
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
Demandes liées
Révisions associées
manager: add minimal search of assets (#11491)
manager: add upload from assets management page (#11491)
manager: add action to overwrite an asset (#11491)
Historique
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.
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).
Mis à jour par Jean-Baptiste Jaillet il y a presque 8 ans
- Fichier
0001-Changes-for-ticket-11491.patchsupprimé
Mis à jour par Jean-Baptiste Jaillet il y a presque 8 ans
- Fichier 0001-Changes-for-ticket-11491.patch 0001-Changes-for-ticket-11491.patch ajouté
- Fichier 0001-manager-change-file-handling-11491.patch 0001-manager-change-file-handling-11491.patch ajouté
- Fichier 0002-manager-Add-pagination-for-assets-11491.patch 0002-manager-Add-pagination-for-assets-11491.patch ajouté
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
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 ?
Mis à jour par Jean-Baptiste Jaillet il y a presque 8 ans
- Fichier 0001-manager-new-handle-for-assets-11491.patch 0001-manager-new-handle-for-assets-11491.patch ajouté
Bon, j'ai fait un grand mic mac avec git pour tout remettre d'aplomb^^
J'ai fait les modifications qu'il restait encore.
Mis à jour par Frédéric Péters il y a environ 7 ans
- Lié à Development #9592: Ajout d'un bloc de contenu de type document ajouté
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 ?
Mis à jour par Jean-Baptiste Jaillet il y a environ 7 ans
- Fichier 0001-manager-new-handle-for-assets-11491.patch 0001-manager-new-handle-for-assets-11491.patch ajouté
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.
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.
Mis à jour par Frédéric Péters il y a plus de 6 ans
- Fichier 0004-manager-add-action-to-overwrite-an-asset-11491.patch 0004-manager-add-action-to-overwrite-an-asset-11491.patch ajouté
- Fichier 0003-manager-add-upload-from-assets-management-page-11491.patch 0003-manager-add-upload-from-assets-management-page-11491.patch ajouté
- Fichier 0002-manager-add-minimal-search-of-assets-11491.patch 0002-manager-add-minimal-search-of-assets-11491.patch ajouté
- Fichier 0001-manager-revamp-assets-management-page-11491.patch 0001-manager-revamp-assets-management-page-11491.patch ajouté
- Patch proposed changé de Non à Oui
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)
Mis à jour par Frédéric Péters il y a plus de 6 ans
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
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.
Mis à jour par Thomas Noël il y a plus de 6 ans
- Fichier
Capture d’écran de 2017-12-29 19-25-56.pngsupprimé
Mis à jour par Thomas Noël il y a plus de 6 ans
- Fichier Capture d’écran de 2017-12-29 19-27-03.png Capture d’écran de 2017-12-29 19-27-03.png ajouté
Oups... la bonne copie d'écran.
Mis à jour par Frédéric Péters il y a plus de 6 ans
- Fichier 0001-manager-revamp-assets-management-page-11491.patch 0001-manager-revamp-assets-management-page-11491.patch ajouté
Voilà, avec tout le temps un tr::before.
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".
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).
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).
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)
Mis à jour par Brice Mallet il y a plus de 6 ans
- Lié à Development #21008: Améliorations possibles sur interface "Ressources" ajouté
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
manager: revamp assets management page (#11491)