-
Notifications
You must be signed in to change notification settings - Fork 4
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
base: main
Are you sure you want to change the base?
Conversation
2c7d1ac
to
f1a5b45
Compare
<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"> |
There was a problem hiding this comment.
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 |
There was a problem hiding this comment.
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 |
There was a problem hiding this comment.
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 ?
There was a problem hiding this comment.
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 ?
There was a problem hiding this comment.
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 ?
There was a problem hiding this 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 !
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.