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.

Sign up for GitHub

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

Jump to bottom

Feature/bao tooltip #158

Open
wants to merge 12 commits into
base: master
Choose a base branch
from
Open

Feature/bao tooltip #158

wants to merge 12 commits into from

Conversation

jfg69
Copy link

@jfg69 jfg69 commented Mar 26, 2023

J'ai refactorisé le code suite aux commentaires de Grégory
J'ai ajusté les styles
Par contre pour ce qui est des styles, certains ne semblent pas bien fonctionner dans storybook ??? Mais ça semble le cas également pour d'autres composantes comme le bouton par exemple. Donc est-ce seulement pour moi ???
J'ai eu quelque soucis avec git mais tout semble rentré dans l'ordre.

@jfg69 jfg69 requested a review from a team as a code owner March 26, 2023 15:35
@jfg69
Copy link
Author

jfg69 commented Mar 26, 2023

J'ai des problèmes avec les signed-off
Je sais pas trop comment corriger ça ?
J'ai essayé mais ça ne semble pas fonctionner.

@OlivierAlbertini
Copy link
Member

J'ai refactorisé le code suite aux commentaires de Grégory

J'ai ajusté les styles

Par contre pour ce qui est des styles, certains ne semblent pas bien fonctionner dans storybook ??? Mais ça semble le cas également pour d'autres composantes comme le bouton par exemple. Donc est-ce seulement pour moi ???

J'ai eu quelque soucis avec git mais tout semble rentré dans l'ordre.

https://services.montreal.ca/bao-storybook/?path=/docs/components-button--primary

Peux tu nous dire qu'est ce qui ne fonctionne pas exactement?

@OlivierAlbertini
Copy link
Member

J'ai des problèmes avec les signed-off

Je sais pas trop comment corriger ça ?

J'ai essayé mais ça ne semble pas fonctionner.

Il faut signer les commits voir https://github.com/VilledeMontreal/angular-ui/blob/master/CONTRIBUTING_FR.md

@jfg69
Copy link
Author

jfg69 commented Mar 26, 2023

J'ai refactorisé le code suite aux commentaires de Grégory
J'ai ajusté les styles
Par contre pour ce qui est des styles, certains ne semblent pas bien fonctionner dans storybook ??? Mais ça semble le cas également pour d'autres composantes comme le bouton par exemple. Donc est-ce seulement pour moi ???
J'ai eu quelque soucis avec git mais tout semble rentré dans l'ordre.

https://services.montreal.ca/bao-storybook/?path=/docs/components-button--primary

Peux tu nous dire qu'est ce qui ne fonctionne pas exactement?

J'ai utilisé le mixin typo-interface-small qui a un font-size de 0,875rem (14px) et le font-size semble être 16px dans le rendu ???
Quand je regardes d'autres composante dans storybook comme le bouton par exemple le font-size devrait aussi être de 14px et moi je vois 16px dans storybook donc c'est pour ça que je crois que le problème est peut-être de mon côté.

MaudeLaflamme and others added 11 commits March 26, 2023 14:30
@MaudeLaflamme @jfg69
fix(file): remove unnecessary margin and fix error when no guiding te…
96b3c60 
…st is provided. close VilledeMontreal#117. (VilledeMontreal#120)

Signed-off-by: Maude LAFLAMME <maude.laflamme@montreal.ca>
(cherry picked from commit a43dc66)
Signed-off-by: Jean-Francois Gamache <jfgamache@hotmail.com>
@TolaSam @jfg69
Feat: Remove animation module
898ef26 
Signed-off-by: TolaSam <tola.sam@montreal.ca>
Signed-off-by: Jean-Francois Gamache <jfgamache@hotmail.com>
@TolaSam @jfg69
14.1.0
8fe34c8 
Signed-off-by: TolaSam <tola.sam@montreal.ca>
Signed-off-by: Jean-Francois Gamache <jfgamache@hotmail.com>
@TolaSam @jfg69
Bump version to 14.1.0
86eadb7 
Signed-off-by: TolaSam <tola.sam@montreal.ca>
Signed-off-by: Jean-Francois Gamache <jfgamache@hotmail.com>
@MaudeLaflamme @jfg69
fix(dropdown-menu): fix menu does not close when menu item is clicked. …
9b078e2 
…close VilledeMontreal#124. (VilledeMontreal#125) (VilledeMontreal#127)

Signed-off-by: Maude LAFLAMME <maude.laflamme@montreal.ca>
(cherry picked from commit 8ce477d)
Signed-off-by: Jean-Francois Gamache <jfgamache@hotmail.com>
@Ahryman40k @jfg69
feat: Update to angular v15 (VilledeMontreal#130)
9e601a3 
* feat:  Update to angular v15

Signed-off-by: Geoffrey Baudin <geoffrey.baudin.ext@montreal.ca>

* chore(tsconfig): fix target value to es2018 (was ES2022)

Signed-off-by: Geoffrey Baudin <geoffrey.baudin.ext@montreal.ca>

* chore(package.json): manually fix version

Signed-off-by: Geoffrey Baudin <geoffrey.baudin.ext@montreal.ca>

* docs(readme): add angular 15 support

Signed-off-by: Geoffrey Baudin <geoffrey.baudin.ext@montreal.ca>

* fix test discovery in karma

Signed-off-by: Geoffrey Baudin <geoffrey.baudin.ext@montreal.ca>

* chore: lint

Signed-off-by: Geoffrey Baudin <geoffrey.baudin.ext@montreal.ca>

* chore: package.json

Signed-off-by: Geoffrey Baudin <geoffrey.baudin.ext@montreal.ca>

---------

Signed-off-by: Geoffrey Baudin <geoffrey.baudin.ext@montreal.ca>
Co-authored-by: Geoffrey Baudin <geoffrey.baudin.ext@montreal.ca>
Signed-off-by: Jean-Francois Gamache <jfgamache@hotmail.com>
@jfg69
Add tooltip directive/component
0352b1d 
Signed-off-by: Jean-francois Gamache <jean-francois.gamache@montreal.ca>
Signed-off-by: Jean-Francois Gamache <jfgamache@hotmail.com>
@jfg69
fixe positionning on DOM
09350fc 
refacto code
Signed-off-by: Jean-francois Gamache <jean-francois.gamache@montreal.ca>

Signed-off-by: Jean-Francois Gamache <jfgamache@hotmail.com>
@mtlmatt @jfg69
feat: VilledeMontreal#54 reintroduce the snackbar (VilledeMontreal#151)
f76c52f 
* feat: VilledeMontreal#54 reintroduce the snackbar

Signed-off-by: Matthieu <matthieu.perrin@montreal.ca>

* fix: code review changes

Signed-off-by: Matthieu <matthieu.perrin@montreal.ca>

---------

Signed-off-by: Matthieu <matthieu.perrin@montreal.ca>
Signed-off-by: Jean-Francois Gamache <jfgamache@hotmail.com>
@mtlmatt @jfg69
ci(release): v15.1.0
2d67465 
Signed-off-by: Matthieu <matthieu.perrin@montreal.ca>
Signed-off-by: Jean-Francois Gamache <jfgamache@hotmail.com>
@jfg69
fix styling
75e9319 
Signed-off-by: Jean-francois Gamache <jean-francois.gamache@montreal.ca>

Signed-off-by: Jean-Francois Gamache <jfgamache@hotmail.com>
@jfg69 jfg69 force-pushed the feature/bao-tooltip branch from 0e94aa8 to 75e9319 Compare March 26, 2023 18:31
@jfg69
Copy link
Author

jfg69 commented Mar 27, 2023

J'ai des problèmes avec les signed-off
Je sais pas trop comment corriger ça ?
J'ai essayé mais ça ne semble pas fonctionner.

Il faut signer les commits voir https://github.com/VilledeMontreal/angular-ui/blob/master/CONTRIBUTING_FR.md

Finalement j'ai réussi à corriger et ajouter les signatures

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
6 participants
@jfg69 @OlivierAlbertini @MaudeLaflamme @TolaSam @Ahryman40k @mtlmatt