Development #18599
Avoir une validation côté client de la taille max pour les fichiers
0%
Description
Pour un retour vers l'usager avant même le transfert. (il y a la même demande côté w.c.s., #13395).
Fichiers
Demandes liées
Révisions associées
Historique
Mis à jour par Paul Marillonnet il y a plus de 6 ans
- Fichier 0001-WIP-frontend-document-upload-size-validation-18599.patch 0001-WIP-frontend-document-upload-size-validation-18599.patch ajouté
- Patch proposed changé de Non à Oui
Je commence un peu à jouer avec jQuery, outil avec lequel je ne suis pas familier (cf patch WIP qui détermine côté client la taille du fichier chargé).
Je pense poser un patch fonctionnel aujourd'hui ou demain.
Mis à jour par Paul Marillonnet il y a plus de 6 ans
- Fichier 0001-WIP-frontend-document-upload-size-validation-18599.patch 0001-WIP-frontend-document-upload-size-validation-18599.patch ajouté
Voilà j'ai bricolé un bout de JS qui fait le boulot. Je m'occupe de l'i18n et des traductions.
Mis à jour par Frédéric Péters il y a plus de 6 ans
var filesize = this.files0.size, max_document_size = {{ max_document_size }};
Sur la forme, déclare chaque variable sur une ligne propre.
$('#send-file input[type=submit]').click();
Et sur la forme aussi, cette ligne est légèrement moins indentée que les autres.
Mis à jour par Paul Marillonnet il y a plus de 6 ans
- Fichier 0001-French-translations-for-18599.patch 0001-French-translations-for-18599.patch ajouté
- Fichier 0001-frontend-document-upload-size-validation-18599.patch 0001-frontend-document-upload-size-validation-18599.patch ajouté
Avec l'i18n et les traductions.
J'affiche aussi la taille en ko (à vrai dire en kibioctet pour les puristes) plutot qu'en octets.
Mis à jour par Frédéric Péters il y a plus de 6 ans
J'affiche aussi la taille en ko (à vrai dire en kibioctet pour les puristes) plutot qu'en octets.
Tape plutôt directement en Mo, qui fournira des nombres plus lisibles.
Dans les processus de traduction, ça ne se fait pas de composer des phrases avec des petits morceaux. Sur le texte même, ça parle de documents et de fichiers et au pluriel, il ne faut utiliser un seul terme dans l'interface, et au singulier.
Mis à jour par Paul Marillonnet il y a plus de 6 ans
- Fichier 0001-WIP-frontend-document-upload-size-validation-18599.patch 0001-WIP-frontend-document-upload-size-validation-18599.patch ajouté
Beh du coup c'est parti sur un filtre de balise perso.
De nouveau un patch 'WIP', il faut que je teste bien tout en détail et que je mette les traductions.
Mis à jour par Frédéric Péters il y a plus de 6 ans
Beh du coup c'est parti sur un filtre de balise perso.
https://docs.djangoproject.com/en/1.8/ref/templates/builtins/#filesizeformat doit faire le taf.
Mis à jour par Paul Marillonnet il y a plus de 6 ans
- Fichier 0001-WIP-frontend-document-upload-size-validation-18599.patch 0001-WIP-frontend-document-upload-size-validation-18599.patch ajouté
Plus simple comme ça en effet.
Mis à jour par Paul Marillonnet il y a plus de 6 ans
- Lié à Development #20139: màj i18n et traductions ajouté
Mis à jour par Paul Marillonnet il y a plus de 6 ans
Avec les traductions dans le ticket lié #20139.
Mis à jour par Frédéric Péters il y a plus de 6 ans
Le patch a besoin d'un nouvel intitulé s'il n'est plus work in progress.
Mis à jour par Paul Marillonnet il y a plus de 6 ans
- Fichier 0001-frontend-document-upload-size-validation-18599.patch 0001-frontend-document-upload-size-validation-18599.patch ajouté
Oui c'est vrai, voilà c'est corrigé, merci.
Mis à jour par Frédéric Péters il y a plus de 6 ans
- Fichier 0001-add-browser-side-validation-of-document-size-18599.patch 0001-add-browser-side-validation-of-document-size-18599.patch ajouté
Désolé j'ai ensuite oublié; je viens de le tester et je trouve qu'on ne voit pas assez l'erreur, du coup j'ai commencé par la déplacer au-dessus mais c'est un peu usant de viser juste avec jquery donc tout simplement j'ai préféré ajouter le texte à l'endroit désiré et le faire apparaitre au moment opportun.
Aussi, j'aurais bien touché au style du message mais le fargo/css/style.css n'est de toute façon pas repris côté thème Publik, je n'ai du coup pas insisté.
J'attache ma version modifiée de ton patch.
Mis à jour par Paul Marillonnet il y a plus de 6 ans
- Fichier 0001-add-browser-side-validation-of-document-size-18599.patch 0001-add-browser-side-validation-of-document-size-18599.patch ajouté
Frédéric Péters a écrit :
Désolé j'ai ensuite oublié; je viens de le tester et je trouve qu'on ne voit pas assez l'erreur, du coup j'ai commencé par la déplacer au-dessus mais c'est un peu usant de viser juste avec jquery donc tout simplement j'ai préféré ajouter le texte à l'endroit désiré et le faire apparaitre au moment opportun.
Ok d'ac.
Aussi, j'aurais bien touché au style du message mais le fargo/css/style.css n'est de toute façon pas repris côté thème Publik, je n'ai du coup pas insisté.
Peut-être en réutilisant le style gadjo pour les balises span de classe "error-message" ? (cf patch joint)
Mis à jour par Paul Marillonnet il y a plus de 6 ans
Dicussion de vive voix avec Frédéric sur mon dernier patch et sur la place actuelle de gadjo dans publik.
Du coup Frédéric, ack pour ton patch. Merci pour ton aide.
Mis à jour par Paul Marillonnet il y a plus de 6 ans
- Statut changé de En cours à Résolu (à déployer)
commit 0aeef5bc1ff6de00372d0e9ea8c4b02a2b7304d3
Author: Paul Marillonnet <pmarillonnet@entrouvert.com>
Date: Thu Nov 16 15:55:16 2017 +0100
add browser-side validation of document size (#18599)
Mis à jour par Benjamin Dauvergne il y a environ 6 ans
- Statut changé de Résolu (à déployer) à Fermé
add browser-side validation of document size (#18599)