entrouvert/wcs
entrouvert
/
wcs
14
1
Fork 0
Code Issues Pull Requests 34 Releases Activity

produire des vignettes pour les fichiers SVG (#75505) #176

Merged
fpeters merged 2 commits from wip/75505-La-production-de-vignettes-pour into main 2023-03-31 10:34:41 +02:00
Conversation 4 Commits 2 Files Changed 4 +55 -7
bdauvergne commented 2023-03-20 17:45:30 +01:00
Owner
Copy Link
No description provided.
bdauvergne added 1 commit 2023-03-20 17:45:30 +01:00
fpeters requested changes 2023-03-20 17:53:04 +01:00
fpeters left a comment
Owner
Copy Link

J'aurais plutôt, comme combo que je citais dans le ticket, retourné le fichier de base même quand ?thumbnail est passé.

J'aurais plutôt, comme combo que je citais dans le ticket, retourné le fichier de base même quand ?thumbnail est passé.
bdauvergne commented 2023-03-20 18:01:45 +01:00
Author
Owner
Copy Link

J'aurais plutôt, comme combo que je citais dans le ticket, retourné le fichier de base même quand ?thumbnail est passé.

C'est exactement ce que je fais.

> J'aurais plutôt, comme combo que je citais dans le ticket, retourné le fichier de base même quand ?thumbnail est passé. C'est exactement ce que je fais.
bdauvergne requested review from fpeters 2023-03-20 18:01:51 +01:00
fpeters commented 2023-03-20 18:55:10 +01:00
Owner
Copy Link

C'est exactement ce que je fais.

Ok alors j'imaginais que le code pour faire ça aurait été moins moche.

Ça vient en partie par le début avec une ligne qui devient trop longue parce que CSS inline, et le formatage que ça amène. (peut-être plutôt juster poser une classe, qui sera à définir à la fois pour le backoffice et le front, oui).

Ensuite peut-être moche aussi par la répétition de la vérification du content type, peut-être qu'un misc.is_svg(content_type) donnerait mieux; ou peut-être que ça aurait pu être géré en début de méthode serve_file, avoir un if thumbnail and is_svg(...): thumbnail = False.

Et pour finir l'absence de test rend moins évidente la lecture du comportement attendu.

> C'est exactement ce que je fais. Ok alors j'imaginais que le code pour faire ça aurait été moins moche. Ça vient en partie par le début avec une ligne qui devient trop longue parce que CSS inline, et le formatage que ça amène. (peut-être plutôt juster poser une classe, qui sera à définir à la fois pour le backoffice et le front, oui). Ensuite peut-être moche aussi par la répétition de la vérification du content type, peut-être qu'un misc.is_svg(content_type) donnerait mieux; ou peut-être que ça aurait pu être géré en début de méthode serve_file, avoir un `if thumbnail and is_svg(...): thumbnail = False`. Et pour finir l'absence de test rend moins évidente la lecture du comportement attendu.
fpeters refused to review 2023-03-20 18:55:17 +01:00
bdauvergne force-pushed wip/75505-La-production-de-vignettes-pour from 8dfaa57d98 to 633490987c 2023-03-21 08:09:41 +01:00 Compare
bdauvergne commented 2023-03-21 08:10:14 +01:00
Author
Owner
Copy Link

Ok alors j'imaginais que le code pour faire ça aurait été moins moche.

Ok.

> Ok alors j'imaginais que le code pour faire ça aurait été moins moche. Ok.
bdauvergne requested review from fpeters 2023-03-21 08:10:17 +01:00
bdauvergne force-pushed wip/75505-La-production-de-vignettes-pour from 633490987c to b5cf677b61 2023-03-21 11:25:55 +01:00 Compare
fpeters requested changes 2023-03-24 10:17:14 +01:00
wcs/qommon/static/css/sofresh.css
@ -330,3 +330,4 @@ div.icon-important { padding-left: 34px; background-image: url(../images/yelp-no
div.icon-tip { padding-left: 34px; background-image: url(../images/yelp-note-tip.png); }
div.icon-warning { padding-left: 34px; background-image: url(../images/yelp-note-warning.png); }
.file-field--thumbnail { max-height: 300px; max-width: 300px; height: auto; width: auto; }
fpeters commented 2023-03-24 10:17:08 +01:00
Owner
Copy Link

Ce fichier n'est pas utilisé (je vais faire un ticket pour retirer ça); pour le backoffice wcs/qommon/static/css/dc2/admin.scss, pour le front c'est dans publik-base-theme.

Ce fichier n'est pas utilisé (je vais faire un ticket pour retirer ça); pour le backoffice wcs/qommon/static/css/dc2/admin.scss, pour le front c'est dans publik-base-theme.
bdauvergne commented 2023-03-24 10:28:28 +01:00
Author
Owner
Copy Link

Ok je vais ouvrir un ticket coté publik-base-theme.

Ok je vais ouvrir un ticket coté publik-base-theme.
bdauvergne commented 2023-03-24 10:41:47 +01:00
Author
Owner
Copy Link

Finalement ce bout de css et la nouvelle classe n'était pas nécessaires pour le backoffice, la CSS actuelle1 fait l'affaire (j'ai peut-être basé mon observation uniquement sur le front). J'ai retiré les deux. C'est surtout en front qu'il n'y a aucun style sur le tag de la vignette.

div.file-field img {
max-width: 100%;
max-height: 25vh;
display: block;
}

Finalement ce bout de css et la nouvelle classe n'était pas nécessaires pour le backoffice, la CSS actuelle[1] fait l'affaire (j'ai peut-être basé mon observation uniquement sur le front). J'ai retiré les deux. C'est surtout en front qu'il n'y a aucun style sur le tag <img> de la vignette. [1]: ``` div.file-field img { max-width: 100%; max-height: 25vh; display: block; } ```
bdauvergne marked this conversation as resolved
bdauvergne force-pushed wip/75505-La-production-de-vignettes-pour from b5cf677b61 to 78bd6cb136 2023-03-24 10:25:47 +01:00 Compare
bdauvergne force-pushed wip/75505-La-production-de-vignettes-pour from 78bd6cb136 to 6b534306b5 2023-03-24 10:28:08 +01:00 Compare
bdauvergne requested review from fpeters 2023-03-24 10:33:19 +01:00
bdauvergne force-pushed wip/75505-La-production-de-vignettes-pour from 6b534306b5 to d338bdbf1f 2023-03-24 10:39:11 +01:00 Compare
fpeters commented 2023-03-29 12:35:00 +02:00
Owner
Copy Link

J'ai créé entrouvert/publik-base-theme#209 pour la modification côté publik-base-theme.

J'ai créé https://git.entrouvert.org/entrouvert/publik-base-theme/pulls/209 pour la modification côté publik-base-theme.
fpeters approved these changes 2023-03-31 10:34:30 +02:00
fpeters left a comment
Owner
Copy Link

Voilà le côté publik-base-theme a été intégré, go pour ici.

Voilà le côté publik-base-theme a été intégré, go pour ici.
fpeters merged commit 6d11417ae3 into main 2023-03-31 10:34:41 +02:00
fpeters deleted branch wip/75505-La-production-de-vignettes-pour 2023-03-31 10:34:41 +02:00
Sign in to join this conversation.
Reviewers
No reviewers
fpeters
Labels
Clear labels
No items
No Label
Milestone
Clear milestone
No items
No Milestone
Assignees
Clear assignees
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: entrouvert/wcs#176
No description provided.
Delete Branch "wip/75505-La-production-de-vignettes-pour"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?