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

♻️(frontend) replace grommet DropButton with custom DropButton #2513

Merged
merged 3 commits into from
Nov 27, 2023

Conversation

AntoLC
Copy link
Contributor

@AntoLC AntoLC commented Nov 23, 2023

Purpose

In order to do the transition to the cunningham design system,
we replace the grommet DropButton component with a custom
DropButton components based on the Popover cunningham component.

Proposal

  • Replace some Select with the Cunningham Select
  • Replace DropButton

Visual change

Dropdown white, icon blue

image

Remove the footer wave on small device on login page

image

@AntoLC AntoLC self-assigned this Nov 23, 2023
@AntoLC AntoLC force-pushed the refacto/anthony/grommet-select-dropButton branch 3 times, most recently from b889744 to 822b37d Compare November 24, 2023 11:11
We replace the grommet Select components by the cunningham
Select components.
The footer was stepping on the login form on some devices. This commit
fixes that.
@AntoLC AntoLC force-pushed the refacto/anthony/grommet-select-dropButton branch from 822b37d to a5f417d Compare November 24, 2023 13:11
In order to do the transition to the cunningham design system,
we replace the grommet DropButton component with a custom DropButton
components based on the Popover cunningham component.
@AntoLC AntoLC force-pushed the refacto/anthony/grommet-select-dropButton branch from a5f417d to cef0862 Compare November 24, 2023 14:06
@AntoLC AntoLC marked this pull request as ready for review November 24, 2023 14:13
@AntoLC AntoLC merged commit 89546f6 into master Nov 27, 2023
33 of 34 checks passed
@AntoLC AntoLC deleted the refacto/anthony/grommet-select-dropButton branch November 27, 2023 08:55
lunika added a commit that referenced this pull request Nov 29, 2023
Added

- Meta description and meta title on the website from the API (#2516)
- Retrieve BBB learning analytics and send them through API
- Classroom attendance analytics (#2499)
- Add a language picker for the invite link on the website (#2504)

Changed

- Replace grommet DropButton component (#2513)
- On live, can now share resource when one is already shared (#2512)
- Remove persistency on token from invite link (#2505)
- Replace grommet Cards / Footer/ Anchor / Tip / Nav (#2503)
- Refacto widgets SharedLiveMedia (#2504)
- Replace grommet Button (#2453)
- Replace grommet Box (#2484)
- Replace grommet TextArea (#2500)
- Update psycopg to version 3
- Update node to version 20
- Replace pylti with oauthlib for LTI request validation
- Replace faker with @faker-js/faker (frontend)

Fixed

- Blocking error when shared document is deleted (#2504)
- Improve render sharing picture in live (#2508)
lunika added a commit that referenced this pull request Nov 29, 2023
Added

- Meta description and meta title on the website from the API (#2516)
- Retrieve BBB learning analytics and send them through API
- Classroom attendance analytics (#2499)
- Add a language picker for the invite link on the website (#2504)

Changed

- Replace grommet DropButton component (#2513)
- On live, can now share resource when one is already shared (#2512)
- Remove persistency on token from invite link (#2505)
- Replace grommet Cards / Footer/ Anchor / Tip / Nav (#2503)
- Refacto widgets SharedLiveMedia (#2504)
- Replace grommet Button (#2453)
- Replace grommet Box (#2484)
- Replace grommet TextArea (#2500)
- Update psycopg to version 3
- Update node to version 20
- Replace pylti with oauthlib for LTI request validation
- Replace faker with @faker-js/faker (frontend)

Fixed

- Blocking error when shared document is deleted (#2504)
- Improve render sharing picture in live (#2508)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants