Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

simplified version removed #711

Merged
merged 3 commits into from
May 6, 2024
Merged

Conversation

xentenza
Copy link
Collaborator

  • "Simplified version" tab removed from news-detail

The project has been abandoned.

- "Simplified version" tab removed from news-detail

The project has been abandoned.
Copy link

github-actions bot commented Mar 26, 2024

Unit Test Results

    1 files      1 suites   0s ⏱️
268 tests 262 ✔️ 0 💤 0  6 🔥
268 runs  256 ✔️ 0 💤 6  6 🔥

For more details on these errors, see this check.

Results for commit e70edfa.

♻️ This comment has been updated with latest results.

Copy link

github-actions bot commented Mar 26, 2024

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


<div class="tab-content py-3">
<div class="tab-pane fade show active" id="original" role="tabpanel" aria-labelledby="original-tab">
<div="news-content-text">
Copy link
Member

Choose a reason for hiding this comment

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

Ce <div> semble causer des soucis au niveau des colonnes :

Screenshot from 2024-03-26 16-14-28

<div class="tab-pane fade show active" id="original" role="tabpanel" aria-labelledby="original-tab">
<div="news-content-text">

<span itemprop="description">
Copy link
Member

Choose a reason for hiding this comment

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

Le <span> est bizarre dans ce contexte avec <p> comme enfants.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Le <span> est bizarre dans ce contexte avec <p> comme enfants.

@williambelle Je suis d'accord avec toi, j'avais juste recopié ce qui figure actuellement sur actu.epfl.ch
Que penses-tu de ma nouvelle PR? ( itemprop="articleBody" )

Copy link
Member

Choose a reason for hiding this comment

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

Oui, c'est mieux.

- corrected schema.org properties
@@ -25,20 +25,11 @@

<div class="container">
<div class="row">
<div class="col-md-6">
<div class="col-md-6 ts-news-content">
Copy link
Member

Choose a reason for hiding this comment

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

Juste supprimer la classe ts-news-content qui n'est pas nécessaire dans le styleguide. (On l'utilise pour les tests).

- deleted ts-news-content
@williambelle williambelle merged commit 69711ca into dev May 6, 2024
4 checks passed
@williambelle williambelle deleted the styleguide/news-simplified-version branch May 6, 2024 09:58
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