Skip to content

Conversation

@MaudeLaflamme
Copy link
Collaborator

Capture d’écran, le 2023-05-24 à 10 09 57

@MaudeLaflamme MaudeLaflamme requested a review from a team as a code owner May 24, 2023 14:29
Copy link
Contributor

@gregpetit gregpetit left a comment

Choose a reason for hiding this comment

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

@MaudeLaflamme

  • La style du texte des infos supplémentaires n'est pas le bon. Ça devrait être le style "Small-Regular (font-size: 14px ; line-height : 20px)
  • Sur mobile, l'info supplémentaire devrait se retrouver sous le tag, avec 4px d'espacement entre les deux (le point de séparation n'est plus présent).

@MaudeLaflamme MaudeLaflamme force-pushed the feature/add-system-header branch from a8873d1 to 77dc891 Compare May 25, 2023 13:12
@MaudeLaflamme
Copy link
Collaborator Author

@gregpetit
C'est noté pour le style du texte d'info supplémentaires.
Pour ton deuxième point, tu veux dire que lorsque tu passes à mobile, l'info supplémentaire ne va pas sous le tag?
image

Copy link
Member

@OlivierAlbertini OlivierAlbertini left a comment

Choose a reason for hiding this comment

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

LGTM

@gregpetit gregpetit self-requested a review May 26, 2023 18:10


Signed-off-by: Maude Laflamme <maude.laflamme@montreal.ca>
@MaudeLaflamme MaudeLaflamme force-pushed the feature/add-system-header branch from fead478 to 877fc5b Compare May 26, 2023 18:33
@MaudeLaflamme MaudeLaflamme merged commit 4aca555 into VilledeMontreal:master May 26, 2023
@MaudeLaflamme MaudeLaflamme deleted the feature/add-system-header branch May 26, 2023 18:38
mtlmatt pushed a commit that referenced this pull request Aug 4, 2023
Signed-off-by: Maude Laflamme <maude.laflamme@montreal.ca>
mtlmatt pushed a commit to mtlmatt/angular-ui that referenced this pull request Aug 4, 2023
 (VilledeMontreal#171)

Signed-off-by: Maude Laflamme <maude.laflamme@montreal.ca>
# Conflicts:
#	projects/angular-ui/src/public-api.ts
mtlmatt pushed a commit to mtlmatt/angular-ui that referenced this pull request Aug 4, 2023
 (VilledeMontreal#171)

Signed-off-by: Maude Laflamme <maude.laflamme@montreal.ca>
mtlmatt pushed a commit to mtlmatt/angular-ui that referenced this pull request Aug 4, 2023
 (VilledeMontreal#171)

Signed-off-by: Maude Laflamme <maude.laflamme@montreal.ca>
mtlmatt pushed a commit to mtlmatt/angular-ui that referenced this pull request Aug 4, 2023
 (VilledeMontreal#171)

Signed-off-by: Maude Laflamme <maude.laflamme@montreal.ca>
mtlmatt pushed a commit to mtlmatt/angular-ui that referenced this pull request Aug 4, 2023
 (VilledeMontreal#171)

Signed-off-by: Maude Laflamme <maude.laflamme@montreal.ca>
mtlmatt pushed a commit to mtlmatt/angular-ui that referenced this pull request Aug 4, 2023
 (VilledeMontreal#171)

Signed-off-by: Maude Laflamme <maude.laflamme@montreal.ca>
mtlmatt pushed a commit to mtlmatt/angular-ui that referenced this pull request Aug 4, 2023
 (VilledeMontreal#171)

Signed-off-by: Maude Laflamme <33531625+MaudeLaflamme@users.noreply.github.com>
mtlmatt pushed a commit to mtlmatt/angular-ui that referenced this pull request Aug 4, 2023
 (VilledeMontreal#171)

Signed-off-by: MaudeLaflamme <33531625+MaudeLaflamme@users.noreply.github.com>
mtlmatt pushed a commit that referenced this pull request Aug 4, 2023
Signed-off-by: MaudeLaflamme <33531625+MaudeLaflamme@users.noreply.github.com>
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.

3 participants