rust-lang/this-week-in-rust

Header formatting on email version of TWiR is not line breaking correctly

Open

#3 592 ouverte le 26 août 2022

Voir sur GitHub
 (10 commentaires) (0 réactions) (0 assignés)HTML (1 010 forks)batch import
bughelp wanted

Métriques du dépôt

Stars
 (1 932 stars)
Métriques de merge PR
 (Merge moyen 1j 17h) (113 PRs mergées en 30 j)

Description

The title image and header of the email are not breaking correctly in the generated HTML code for the TwiR email issue.

The break should be placed after the "This Week in Rust" title image. Instead, the line is currently breaking after the "This" in the text header.

The email code is being generated by the optimize-email Makefile job, which uses the website code (via the make build && make generate-website && make host-content command) to create an email using juice.

<td style="font-family: 'Helvetica Neue', Helvetica, sans-serif;">
  <img class="wrapper-image " src="https://this-week-in-rust.org/images/logo32.png" alt="This Week in Rust" style="width: 30px; vertical-align: bottom; max-width: 100%;" width="30">
  <span class="wrapper-header-01" style="line-height: 27px; font-size: 23px; font-weight: bold; color: #444444;">This Week in Rust</span>
</td>

Narrow window: Screen Shot 2022-08-25 at 6 25 39 PM

Wide window: Screen Shot 2022-08-25 at 6 23 48 PM

Guide contributeur

Direction de recherche
Examinez le code de génération d'e mail dans le dépôt, en particulier la cible `optimize email` du Makefile et le modèle HTML associé. Recherchez les styles en ligne qui contrôlent les éléments img et span. Envisagez d'ajouter un saut de ligne clair (par exemple un <br> ou en utilisant CSS display block) après l'image pour garantir un saut de ligne correct. Testez avec différentes largeurs d'e mail pour vérifier la correction.
Stack technique
htmlcss
Domaine
frontend
Type d'issue
Bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
Moins d'une heure
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Ancienne
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
CSS basicsHTML email rendering
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
75