entrouvert
/
publik-base-theme
15
1
Fork 1
Code Issues Pull Requests 9 Releases Wiki Activity

emails: add class to center images (#82016) #406

Merged
smihai merged 1 commits from wip/82016-emails-add-class-to-center-images into main 2023-11-06 15:55:00 +01:00
Conversation 0 Commits 1 Files Changed 1 +4
smihai commented 2023-10-05 10:09:55 +02:00
Owner
Copy Link
No description provided.
smihai added 1 commit 2023-10-05 10:09:55 +02:00
gitea/publik-base-theme/pipeline/head There was a failure building this commit Details
3d637a3e8a emails: add class to center images (#82016)
smihai force-pushed wip/82016-emails-add-class-to-center-images from 3d637a3e8a to 0dc6ab2030 2023-10-05 10:17:40 +02:00 Compare
fpeters reviewed 2023-10-05 10:18:32 +02:00
templates/emails/body_base.html
@ -79,6 +79,10 @@
background: #555555 !important;
border-color: #555555 !important;
}
img.align-center {
fpeters commented 2023-10-05 10:18:32 +02:00
Owner
Copy Link

C'est peut-être plus compliqué que ça si on veut quelque chose qui fonctionne dans les différents clients mails; ça demanderait des tests.

C'est peut-être plus compliqué que ça si on veut quelque chose qui fonctionne dans les différents clients mails; ça demanderait des tests.
bdauvergne commented 2023-11-06 15:03:32 +01:00
Owner
Copy Link

À regarder une liste de compatibilité ça m'a l'air ok, https://www.campaignmonitor.com/css/positioning-display/display/ & https://www.campaignmonitor.com/css/box-model/margin/ . Les principaux webmail et lecteurs desktop sont supportés. Au niveau de la structure on est sûr d'avoir toujours un tag img dans un tag div.document qui est bien un block. Et c'est la solution proposé sur stackoverflow pour centrer un bloc dans un email.

Aussi testé en live sur gmail ça passe aussi (j'ai pas de compte live.com ou yahoo.com mais je peux forwarder le mail).

À regarder une liste de compatibilité ça m'a l'air ok, https://www.campaignmonitor.com/css/positioning-display/display/ & https://www.campaignmonitor.com/css/box-model/margin/ . Les principaux webmail et lecteurs desktop sont supportés. Au niveau de la structure on est sûr d'avoir toujours un tag img dans un tag div.document qui est bien un block. Et c'est la solution proposé sur stackoverflow pour centrer un bloc dans un email. Aussi testé en live sur gmail ça passe aussi (j'ai pas de compte live.com ou yahoo.com mais je peux forwarder le mail).
smihai commented 2023-11-06 15:54:54 +01:00
Author
Owner
Copy Link

J'avoue n'avoir testé que sous Roundcube et le rendu est ok.

Merci pour le boulot de test sur les autres mails.

J'avoue n'avoir testé que sous Roundcube et le rendu est ok. Merci pour le boulot de test sur les autres mails.
bdauvergne approved these changes 2023-11-06 15:03:42 +01:00
bdauvergne left a comment
Owner
Copy Link

Ok.

Ok.
smihai merged commit 04d38899df into main 2023-11-06 15:55:00 +01:00
smihai deleted branch wip/82016-emails-add-class-to-center-images 2023-11-06 15:55:00 +01:00
Sign in to join this conversation.
Reviewers
No reviewers
bdauvergne
Labels
Clear labels
No items
No Label
Milestone
Clear milestone
No items
No Milestone
Assignees
Clear assignees
No Assignees
3 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/publik-base-theme#406
No description provided.
Delete Branch "wip/82016-emails-add-class-to-center-images"

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?