Skip to content

Conversation

@xentenza
Copy link
Collaborator

Ajout d'un template pour le site des revues de presse rdp.epfl.ch

Ajout d'un template pour le site des revues de presse rdp.epfl.ch
@github-actions
Copy link

github-actions bot commented Aug 15, 2024

Test Results

  1 files    1 suites   0s ⏱️
279 tests 262 ✅ 0 💤  0 ❌ 17 🔥
279 runs  245 ✅ 0 💤 17 ❌ 17 🔥

For more details on these errors, see this check.

Results for commit b591035.

♻️ This comment has been updated with latest results.

@github-actions
Copy link

github-actions bot commented Aug 15, 2024

🔎 Download the Backstop report for this pull request (link valid for 90 days):

Solution plus élégante pour l'intégration de classes supplémentaires dans "press-review" (distinguer le "content-type" de l'élément intégré dans une page)
Comment on lines +72 to +73
</div>
</div>
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Le nombre de </div> me semble bizarre. J'ai vu la même chose dans d'autres templates.
Je vais essayer de comprendre pourquoi. On peut laisser comme ça en attendant.

</a>
</div>
</div>
<div class="col-md-8">
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Avec une tablette, l'espace n'est pas complétement utilisé. J'avais utilisé col-md-7 col-lg-8 dans rdp.epfl.ch mais tu souhaites peut-être obtenir quelque chose de différent ?

Screenshot from 2024-08-16 14-26-15

@williambelle williambelle merged commit b31ec9f into dev Aug 27, 2024
@williambelle williambelle deleted the styleguide/press-review branch August 27, 2024 16:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants