Skip to content

Conversation

oumeimaelisbihani
Copy link
Contributor

@oumeimaelisbihani oumeimaelisbihani commented Aug 7, 2024

  • Utilisation du même composant ProjectButton pour les projets liés (détails d'un projet) et les cartes thématiques
  • toute la carte est cliquable + tag projet avec lien (utilisation de tabIndex=0 pour forcer l'accessibilité)
  • shadow on active pour la version mobile sur la carte complete

Copy link
Contributor

@dolemoine dolemoine left a 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.

@oumeimaelisbihani oumeimaelisbihani requested review from yvalentin and removed request for yvalentin September 23, 2024 13:53
Copy link
Contributor

@yvalentin yvalentin left a 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 class theme-card sur la balise contenant fr-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 {
...

Copy link
Contributor

@yvalentin yvalentin left a 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. ;-)

@oumeimaelisbihani oumeimaelisbihani merged commit 53b8258 into main Oct 2, 2024
9 checks passed
@oumeimaelisbihani oumeimaelisbihani deleted the feat_project_tag_objective_card branch October 2, 2024 15:05
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.

Ajouter les tags sur les cards objectives de la partie questionnaire
4 participants