Skip to content

feat(topic): featured topics #738

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

Open
wants to merge 4 commits into
base: main
Choose a base branch
from
Open

feat(topic): featured topics #738

wants to merge 4 commits into from

Conversation

abulte
Copy link
Contributor

@abulte abulte commented Apr 24, 2025

Fix ecolabdata/ecospheres#399
Depends opendatateam/udata#3301

Les pastilles sont légèrement différentes vs la maquette, il s'agit d'un DsfrTag standard.

Capture d’écran 2025-04-24 à 10 40 57
Capture d’écran 2025-04-24 à 10 12 34

<h2>Les bouquets à découvrir</h2>
<div class="fr-grid-row fr-grid-row--middle justify-between">
<h2>Les bouquets à découvrir</h2>
<div class="fr-col-auto fr-grid-row fr-mb-3w">
Copy link
Contributor

Choose a reason for hiding this comment

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

Il faudrait pas un <ul class="fr-tags-group"> ici ?

@@ -302,6 +315,18 @@ watch(
class="fr-mb-1v"
@click="togglePublish"
/>
<DsfrButton
Copy link
Contributor

Choose a reason for hiding this comment

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

À mettre en secondary

@@ -302,6 +315,18 @@ watch(
class="fr-mb-1v"
@click="togglePublish"
/>
<DsfrButton
Copy link
Contributor

Choose a reason for hiding this comment

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

IMO on ne veut mettre en avant que les bouquets publiés, pas les brouillons.

On peut par contre préserver featured lors d'une dépublication pour que le bouquet soit "remis" en avant lorsqu'il est republié ? Ou bien on met featured à false lorsqu'il est dépublié, ce qui oblige l'utilisateur à remettre explicitement en avant. J'ai l'impression que la 1e option est un peu plus intuitive que la 2e ?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Oui. Mais ne faut-il pas garder le bouton "Ne pas mettre en avant" si on ne veut plus mettre en avant (à son retour en publié) un bouquet qui est repassé au statut brouillon ?

Copy link
Contributor

Choose a reason for hiding this comment

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

Potentiellement oui, et donc dans ce cas il faudrait garder mettre en avant pour rester logique.

Mais je crains qu'un utilisateur qui clique "mettre en avant" sur un brouillon soit surpris de ne pas le voir mis en avant immédiatement.

On tente comme ça et on voit à l'usage ?

Copy link
Contributor

@Thesauruv Thesauruv left a comment

Choose a reason for hiding this comment

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

Tests OK du point de vue fonctionnel. Merci @abulte !

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