Skip to content

feat : adds primevue#209

Merged
QuentinMadura merged 12 commits intodevfrom
feat/primevue
Apr 25, 2025
Merged

feat : adds primevue#209
QuentinMadura merged 12 commits intodevfrom
feat/primevue

Conversation

@QuentinMadura
Copy link
Contributor

@QuentinMadura QuentinMadura commented Apr 23, 2025

Close #170

Copy link
Contributor

@Marc-AntoineA Marc-AntoineA left a comment

Choose a reason for hiding this comment

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

Qqs remarques, merci d’avoir fait ça. C’est trèèès efficace en effet !

# git submodule should have master branch
track_submodules: true
remote: origin
- name: Get or update showcase code
Copy link
Contributor

Choose a reason for hiding this comment

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

Ça ne marche visiblement pas mieux. Et d’ailleuurs, peut-être que le problème de base vient du passwd… En tout cas ,je préfère que ta modif ne soit pas dans cette MR pour qu’on puisse merger ta MR rapidement.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Je suis d'accord :)

as="a"
class="absolute bottom-[7%] left-[1%] z-10 underline"
href="https://github.com/TelesCoop/iarbre"
icon="pi pi-github"
Copy link
Contributor

Choose a reason for hiding this comment

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

L’icône n’est pas visible ? 
image

Copy link
Contributor

Choose a reason for hiding this comment

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

Merci !

center: [4.8537684279176645, 45.75773479280862],
// zoom to a level that shows the whole city
zoom: 14,
maxZoom: MAX_ZOOM - 1,
Copy link
Contributor

Choose a reason for hiding this comment

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

Ça ne m’a pas l’air équivalent, en tout cas supprimer le commentaire non ? 


@font-face {
font-family: "Bricolage Grotesque";
src: url("/fonts/BricolageGrotesque-Regular.otf") format("opentype");
Copy link
Contributor

Choose a reason for hiding this comment

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

Est-ce que BricolageGrotestque ne devrait pas être supprimé de public/fonts du coup ?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Bien vu !

</button>
<h3 class="popin-heading">Votre avis compte !</h3>
<p class="popin-text">Partagez-nous vos impressions pour nous aider à améliorer le site :</p>
<Dialog
Copy link
Contributor

Choose a reason for hiding this comment

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

Il y a une animation chelou à l’ouverture de la feedback, le bouton "fermé" a une bordure noire. J’ai l’impression que c’est car le bouton a le focus direct. Alors que le focus devrait être au champ text.

Copy link
Contributor Author

@QuentinMadura QuentinMadura Apr 25, 2025

Choose a reason for hiding this comment

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

Effectivement, le bouton a le focus direct mais uniquement sur Firefox. Je propose d'ignorer pour l'instant

@@ -0,0 +1,35 @@
/* eslint-disable */
// @ts-nocheck
// Generated by unplugin-vue-components
Copy link
Contributor

Choose a reason for hiding this comment

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

J’ai l’impression que ce fichier doit être .gitignoré, et sinon, comment le génère-t-on ? 

Copy link
Contributor Author

@QuentinMadura QuentinMadura Apr 25, 2025

Choose a reason for hiding this comment

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

Non il n'a pas vocation à être .gitignoré. Ça vient remplacer globalement les imports que tu fais dans tes composants. Il se génère automatiquement dès que tu importes un nouveau composant !

Copy link
Contributor

Choose a reason for hiding this comment

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

J’ai tendance à me dire que tout ce qui se génère automatiquement peut-être .gitignoré. Ça changerait qqch qu’il soit .gitignoré ?

Copy link
Contributor

Choose a reason for hiding this comment

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

Merci d’avoir regardé

Copy link
Contributor

Choose a reason for hiding this comment

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

C’est assez efficace en effet ! <3

@QuentinMadura QuentinMadura changed the title Feat : Adds primevue feat : adds primevue Apr 25, 2025
@QuentinMadura QuentinMadura merged commit 72e8726 into dev Apr 25, 2025
8 of 9 checks passed
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.

ETQ designer et devs je veux optimiser la collaboration et la réutilisation de composants

2 participants