-
Notifications
You must be signed in to change notification settings - Fork 12
Feature : Ajout de liens directs vers les projets sur les cartes Thématiques du parcours 'j'ai un projet en tete' #947
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
Conversation
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.
Praise: chouette intégration des projets mis en avant cliquable !
todo: en terme fonctionnel, peut-être qu'il faudrait modifier le curseur au hover sur l'ensemble de la card, car on a l'impression que toute la card est cliquable.
…' into feat_project_tag_objective_card
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.
Praise: super pour la navigation. On peux pas ouvrir les cartes dans un nouvelle onglet mais on on verra plus tard si besoin !
Deux point:
- ul/li
<div class="fr-card__desc">
<ul class="fr-tags-group">
<TeeProjectButton
v-for="project in option.highlightProjects"
:key="project.id"
class="highlighted-project fr-mx-1v fr-my-1v inline-flex"
:color="option.color"
size="sm"
:project="project"
/>
<span
v-if="option.moreThanThree"
class="fr-m-auto fr-ml-0"
>...</span
>
</ul>
</div>
Ici si on utilise un ul
, il doit y avoir un li
, de plus avec le fr-tags-group
, il manque l'utilisation de fr-tag
(et donc le css doit être adapté.
- css/scss
Avec la classtheme-card
sur la balise contenantfr-card
. Il doit être possiblie de tout passer en utilisant uniquement les class dsfr et de les surcharger avec nos besoin...
ex
.fr-card {
&.theme-card {
.fr-card__body {
...
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.
Des petits changements à faire. Et pas besoin de refaire un review après. ;-)
Uh oh!
There was an error while loading. Please reload this page.