entrouvert
/
combo
16
1
Fork 0
Code Issues Pull Requests 9 Releases Wiki Activity

wcs: card cell, display option for file field (#60371) #9

Merged
lguerin merged 1 commits from wip/60371-wcs-card-file-field-option into main 2023-02-17 08:52:06 +01:00
Conversation 0 Commits 1 Files Changed 5 +180 -37
lguerin commented 2022-12-16 10:16:15 +01:00
Owner
Copy Link
No description provided.
lguerin reviewed 2022-12-16 10:17:25 +01:00
combo/apps/wcs/templates/combo/wcs/card-field-value.html
@ -21,1 +24,4 @@
{% else %}
<img alt="" src="{% make_public_url url=value.url|add:'&thumbnail=1' %}">
{% endif %}
{% else %}
lguerin commented 2022-12-16 10:17:25 +01:00
Author
Owner
Copy Link

Block else: c'est le mode sans custo, j'ai remis le comportement précédent:
si image, alors balise img, sinon lien.
Mais peut-être qu'on veut afficher une vignette tout le temps ?

Block else: c'est le mode sans custo, j'ai remis le comportement précédent: si image, alors balise img, sinon lien. Mais peut-être qu'on veut afficher une vignette tout le temps ?
fpeters commented 2023-02-14 11:19:12 +01:00
Owner
Copy Link

Mais peut-être qu'on veut afficher une vignette tout le temps ?

Je pense que oui, tout le temps vignette.

> Mais peut-être qu'on veut afficher une vignette tout le temps ? Je pense que oui, tout le temps vignette.
lguerin commented 2023-02-14 14:23:37 +01:00
Author
Owner
Copy Link

vu sur jabber, incompréhension de ma part sur le mode vignette, je reprends mon code

vu sur jabber, incompréhension de ma part sur le mode vignette, je reprends mon code
lguerin commented 2023-02-14 14:58:50 +01:00
Author
Owner
Copy Link

code revu

code revu
lguerin marked this conversation as resolved
lguerin reviewed 2022-12-16 10:18:12 +01:00
combo/apps/wcs/templates/combo/wcs/card-field-value.html Outdated
@ -21,0 +22,4 @@
{% elif value.content_type|startswith:"image/" %}
<img alt="" src="{% make_public_url url=value.url %}">
{% else %}
<img alt="" src="{% make_public_url url=value.url|add:'&thumbnail=1' %}">
lguerin commented 2022-12-16 10:18:12 +01:00
Author
Owner
Copy Link

Param thumbnail seulement si ce n'est pas une image, pour avoir le comportement précédent avec une image.
Mais peut-être qu'on veut toujours avoir le thumbnail ?

Param thumbnail seulement si ce n'est pas une image, pour avoir le comportement précédent avec une image. Mais peut-être qu'on veut toujours avoir le thumbnail ?
fpeters commented 2023-02-14 11:20:14 +01:00
Owner
Copy Link

(oui thumbnail)

(oui thumbnail)
lguerin commented 2023-02-14 14:58:57 +01:00
Author
Owner
Copy Link

code revu

code revu
lguerin marked this conversation as resolved
lguerin reviewed 2022-12-16 10:20:10 +01:00
combo/apps/wcs/templates/combo/wcs/manager/card-cell-form-display.html Outdated
@ -78,6 +78,15 @@
</select>
</label>
</p>
<p data-dynamic-display-child-of="field_varname" data-dynamic-display-value-in=" {% for field in card_schema.fields %}{% if field.type == 'file' %}{{ field.varname }} {% endif %}{% endfor %} ">
lguerin commented 2022-12-16 10:20:10 +01:00
Author
Owner
Copy Link

Trop compliqué de gérer un affichage conditionnel de la forme: si display_mode est "value" ou "label-and-value" ET si c'est un champ fichier.
Du coup, pour un champ fichier, on affiche toujours l'option "File display mode", même si l'agent a choisi "label" seulement.

Trop compliqué de gérer un affichage conditionnel de la forme: si display_mode est "value" ou "label-and-value" ET si c'est un champ fichier. Du coup, pour un champ fichier, on affiche toujours l'option "File display mode", même si l'agent a choisi "label" seulement.
lguerin force-pushed wip/60371-wcs-card-file-field-option from b2a0baf8a5 to 021ee3d2d2 2023-01-03 18:25:35 +01:00 Compare
lguerin force-pushed wip/60371-wcs-card-file-field-option from 021ee3d2d2 to a3d14e9865 2023-02-01 09:37:57 +01:00 Compare
lguerin force-pushed wip/60371-wcs-card-file-field-option from a3d14e9865 to a751aaacb8 2023-02-02 17:31:32 +01:00 Compare
fpeters reviewed 2023-02-14 11:26:46 +01:00
combo/manager/static/js/combo.manager.js
@ -432,3 +432,3 @@
var sel2 = '[data-dynamic-display-value="' + $(this).val() + '"]';
var sel3 = '[data-dynamic-display-value-in*="' + $(this).val() + '"]';
var sel3 = '[data-dynamic-display-value-in*=" ' + $(this).val() + ' "]';
$(sel1).addClass('field-hidden').hide();
fpeters commented 2023-02-14 11:26:46 +01:00
Owner
Copy Link

C'était quoi le bug sans les espaces ? (dans wcs on a une version de ce code sans espace)

C'était quoi le bug sans les espaces ? (dans wcs on a une version de ce code sans espace)
lguerin commented 2023-02-14 14:22:59 +01:00
Author
Owner
Copy Link

Pour éviter de matcher 'label' et 'label-and-value' si on a écrit data-dynamic-display-value-in*=label par exemple.
Et pour éviter de matcher le varname 'foobar' alors que je veux matcher seulement 'foo':

<p data-dynamic-display-child-of="field_varname" data-dynamic-display-value-in=" {% for field in card_schema.fields %}{% if field.type == 'file' %}{{ field.varname }} {% endif %}{% endfor %} ">
Pour éviter de matcher 'label' et 'label-and-value' si on a écrit `data-dynamic-display-value-in*=label` par exemple. Et pour éviter de matcher le varname 'foobar' alors que je veux matcher seulement 'foo': ``` <p data-dynamic-display-child-of="field_varname" data-dynamic-display-value-in=" {% for field in card_schema.fields %}{% if field.type == 'file' %}{{ field.varname }} {% endif %}{% endfor %} "> ```
lguerin force-pushed wip/60371-wcs-card-file-field-option from a751aaacb8 to 2b7b83f80d 2023-02-14 14:58:32 +01:00 Compare
lguerin requested review from fpeters 2023-02-14 14:59:15 +01:00
lguerin reviewed 2023-02-14 15:01:33 +01:00
combo/apps/wcs/templates/combo/wcs/card-field-value.html
@ -20,1 +19,3 @@
<img alt="" src="{% make_public_url url=value.url %}">
{% if item and item.file_display_mode == 'thumbnail' %}
<a class="pk-card-field-filename" href="{% make_public_url url=value.url %}" download="{{ value.filename }}">
{% if value.thumbnail_url %}
lguerin commented 2023-02-14 15:01:33 +01:00
Author
Owner
Copy Link

Il faut https://dev.entrouvert.org/issues/74511 en face

Il faut https://dev.entrouvert.org/issues/74511 en face
fpeters approved these changes 2023-02-17 08:47:29 +01:00
lguerin merged commit 1e03d6b0d4 into main 2023-02-17 08:52:06 +01:00
lguerin deleted branch wip/60371-wcs-card-file-field-option 2023-02-17 08:52:06 +01: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/combo#9
No description provided.
Delete Branch "wip/60371-wcs-card-file-field-option"

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?