Skip to content

News specific pages - #843

Merged
williambelle merged 95 commits into
epfl-si:devfrom
cruncher:actu/page-homepage
Jul 27, 2026
Merged

News specific pages#843
williambelle merged 95 commits into
epfl-si:devfrom
cruncher:actu/page-homepage

Conversation

@mbi

@mbi mbi commented Jun 17, 2026

Copy link
Copy Markdown
Contributor

This is a full assembly of news homepage, news topic and news detail pages, containing several unmerged (at the time) PRs.

@williambelle @xentenza in terms of naming, we already have a "News homepage" (and "News detail") page. Should this page replace the existing one? If not how should we distinguish / name this new page?

NOTE: This branch also contains minor ajustements and improvements to elements that have already been merged! These stem from:

  1. A review of the three assembled pages with the designer, that highlighted some minor adaptations to the components that were not visible when they're isolated.
  2. Some parameters were added to the templates, to add variations (e.g. margins, variations in font-size, ...)

@williambelle are you OK if we submit these changes as a unique set of changes in this PR, or would like individual PRs?

mbi added 30 commits March 26, 2026 14:07
@mbi mbi changed the title WIP: News specific pages News specific pages Jul 1, 2026
@mbi
mbi marked this pull request as ready for review July 1, 2026 13:04
@mbi

mbi commented Jul 1, 2026

Copy link
Copy Markdown
Contributor Author

@xentenza @williambelle this should be the final PR for review. It covers the three example pages (news homepage, news topic page and news detail page) and a few minor cleanup changes here and there (see my comment in the PR description above) 🙏

@williambelle williambelle left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Je suis OK d'avoir tous ces changements en une seule modification.

On peut garder les versions 2026 des pages et les fusionner un peu plus tard.

Selon moi, la page News topic est inutile.

@@ -1,4 +1,4 @@
<a href="#" class="card card-sm news-card link-trapeze-horizontal">
<a href="#" class="card card-sm news-card link-trapeze-horizontal{% if extra_class %} {{ extra_class }}{% endif %} ">

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

trailing space: {% endif %}">

{% if add_wrapper|default(true) %}
</div>
{% endif %}

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Extra newline.


<section class="col-md-8 offset-md-2 col-lg-6 offset-lg-3 mt-4 px-md-0" itemprop="articleBody">
<p>Depuis <a href="#">la première expérience SETI</a> en 1960, les astronomes ont scruté la Voie lactée à la recherche d’indices de l’existence de civilisations extraterrestres avancées. Ces recherches ont porté sur les ondes radio, les flashs optiques et les signatures thermiques infrarouges.</p>
<p>Jusqu'à présent, ils n'ont rien trouvé de crédible. Ce silence s'explique souvent par le fait que nous n'avons exploré qu'une infime partie du paysage cosmique. Mais que se passerait-il si des signaux avaient bien atteint la Terre sans que nous les ayons 
détectés ?</p>

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Unusual character : 'que nous les ayons 
détectés'.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Unnecessary.

{% block header %}
  {% include '@organisms/header/header-light.twig' with { site_title: 'Actu' } %}
{% endblock %}

is enough.

…ght.twig` across templates. Adjust whitespace and formatting inconsistencies.
@mbi

mbi commented Jul 14, 2026

Copy link
Copy Markdown
Contributor Author

Selon moi, la page News topic est inutile.

On a implémenté cette page parce que @xentenza nous avait demandé de monter un exemple pour chacune des pages designées par @8ratio, et c'est la seule page qui présente le composant molecules/card/card-news-horizontal.twig actuellement. Ceci dit on peut la supprimer, aucun souci!

@williambelle

Copy link
Copy Markdown
Member

Je comprends. Je laisse @xentenza décider.

De notre côté (dév.), elle n'est pas nécessaire. Il n'y a rien de plus par rapport à la homepage et au composant card-news-horizontal.twig.

@xentenza

Copy link
Copy Markdown
Collaborator

La frontière entre News homepage 2026 et News topic est effectivement faible.

J'aimerais presque renommer cette dernière Theme homepage 2026, car il y a quelques petites caractéristiques propres aux futures pages d'accueil des facultés, collège, et des thématiques (p. ex: le titre en haut de page). Je peux ainsi présenter plus facilement les rendus lors de mes séances.

Nous verrons à la limite dans un second temps s'il faut les fusionner.

<p>La Terre est « contactée » si un tel signal traverse notre emplacement dans l'espace. La détection n'est possible que si la source se trouve dans une plage de distance qui permette au signal d’être encore suffisamment puissant pour entrer dans le champ de sensibilité de nos télescopes.</p>
<p>Le scientifique a utilisé une approche statistique bayésienne, qui met à jour les estimations à mesure que de nouvelles informations sont prises en compte, pour relier trois éléments :</p>

{% include '@atoms/figure/figure-news.twig' %}

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Merci @mbi pour le rendu complet de la page. J'approuve l'ensemble du squelette, et demanderai des modifications des atomes, molecules et autres si besoin séparément.

Par exemple, la taille de la légende dans @atoms/figure/figure-news.twig, trop proche du texte de base. En défilant verticalement, mes yeux ont d'abord cru qu'il s'agissait de la suite du texte. A discuter avec :ratio

@mbi

mbi commented Jul 21, 2026

Copy link
Copy Markdown
Contributor Author

J'aimerais presque renommer cette dernière Theme homepage 2026,

Parfait merci @xentenza je renomme la page et on peut volontiers faire un mini-round de révision des composants avec :ratio, il nous reste un peu budget de part et d'autre

Edit: fait 6362a9f

@xentenza
xentenza self-requested a review July 22, 2026 13:12
@williambelle
williambelle merged commit 54a224d into epfl-si:dev Jul 27, 2026
2 of 3 checks passed
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