Skip to content
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

Feat/home last topics #562

Closed
wants to merge 14 commits into from
Closed

Feat/home last topics #562

wants to merge 14 commits into from

Conversation

at-github
Copy link
Contributor

@at-github at-github requested review from abulte and narduin October 28, 2024 13:21
Copy link

netlify bot commented Oct 28, 2024

Deploy Preview for ecospheres ready!

Name Link
🔨 Latest commit 84f8031
🔍 Latest deploy log https://app.netlify.com/sites/ecospheres/deploys/6728dbbb07ba0500080ce0e2
😎 Deploy Preview https://deploy-preview-562--ecospheres.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

Copy link

netlify bot commented Oct 28, 2024

Deploy Preview for meteo-france ready!

Name Link
🔨 Latest commit 84f8031
🔍 Latest deploy log https://app.netlify.com/sites/meteo-france/deploys/6728dbbb8eff170008b97a3a
😎 Deploy Preview https://deploy-preview-562--meteo-france.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@at-github at-github force-pushed the feat/home-last-topics branch from 98ad552 to 10116c7 Compare October 29, 2024 08:40
Copy link
Contributor

@abulte abulte left a comment

Choose a reason for hiding this comment

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

  1. Problème de padding sur les tags de la home
Capture d’écran 2024-10-29 à 12 40 56
  1. Responsive : breaker plus tôt à 2 cartes (vers 1095px) ou mieux gérer le responsive des pastilles données et couverture géo
Capture d’écran 2024-10-29 à 12 42 02
  1. Responsive : diminuer et homogénéiser les vignettes quand la carte est pleine largeur
Capture d’écran 2024-10-29 à 12 46 05
  1. Responsive : rester plus longtemps à 2 cartes (jusqu'à 550px environ ?) — voir ce que ça donne avant avec point 3. traité.
Capture d’écran 2024-10-29 à 12 43 22

:text="stripFromMarkdown(bouquet.description)"
:max-lines="3"
<img
src="../../custom/ecospheres/assets/time.svg"
Copy link
Contributor

Choose a reason for hiding this comment

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

Il n'y. a pas une remix icon qui peut faire l'affaire ?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

1 - Fixer avec line-height
2 - Break plus tôt
3 - Vignette fixé

  • Je ne vois pas d'utilisation de remix icon, depuis mes 1ères pr sur la home j'exporte depuis figma ayant surtout croisé des svg

Copy link
Contributor

Choose a reason for hiding this comment

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

On a des ri- un peu partout, e.g. <VIcon name="ri-add-circle-line" class="fr-mr-1v" />.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Méconnaissance je m'attendais à des class, je vais creuser ça

Copy link
Contributor

Choose a reason for hiding this comment

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

Pour info @at-github : https://vue-ds.fr/guide/icones
C'est en cours de mise à jour mais tu peux les utiliser sans risque tant que c'est du ri-

Copy link
Contributor Author

Choose a reason for hiding this comment

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

J'ai tenté : ri-time-line, ri:time-linesans succès (erreur de syntaxe ? problème de version ?) :

Invalid prop: custom validator check failed for prop

Pourtant bien dans https://icon-sets.iconify.design/?query=time (+filtre remixicon)
Comment je peux connaître la liste des dispos ?

Copy link
Contributor

Choose a reason for hiding this comment

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

Parfois il faut les importer via icons.js, elles ne sont pas toutes dans le pack fourni par le DSFR.

Copy link
Contributor

Choose a reason for hiding this comment

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

Parfois il faut les importer via icons.js, elles ne sont pas toutes dans le pack fourni par le DSFR.

Ça n'existe plus ça dans la maj ! 😭
C'est censé fonctionner directement maintenant mais effectivement tant qu'on n'a pas merge #570 ça va créer des conflits.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Parfois il faut les importer via icons.js, elles ne sont pas toutes dans le pack fourni par le DSFR.

Voici la liste trouvé en important icons.js
"MdDragindicator",
"RiAddCircleLine",
"RiClipboardLine",
"RiDatabase2Line",
"RiFileAddLine",
"RiFileCopy2Line",
"RiLightbulbLine",
"RiLogoutBoxRLine",
"RiMap2Line",
"RiPencilLine"

src/components/bouquets/BouquetCard.vue Outdated Show resolved Hide resolved
src/custom/ecospheres/views/HomeView.vue Outdated Show resolved Hide resolved
@at-github at-github force-pushed the feat/home-last-topics branch from 0db612a to 05e2cc9 Compare October 31, 2024 09:39
@at-github
Copy link
Contributor Author

Pull request move to #574

@at-github at-github closed this Nov 5, 2024
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.

[HOME] - Bloc derniers bouquets
3 participants