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(ui): add burger nav to UI #2584

Merged
merged 5 commits into from
Apr 10, 2020
Merged

feat(ui): add burger nav to UI #2584

merged 5 commits into from
Apr 10, 2020

Conversation

UnbearableBear
Copy link
Contributor

@UnbearableBear UnbearableBear commented Apr 7, 2020

GROS CLEAN
J'ai viré un package que j'avais utilisé un peu à contrecoeur. J'ai repris les modals qu'on utilisait déjà. Résultats: Plus d'accessibilité, moins de styles, moins de js, moins de bugs !
Et désormais le burger est accessible dans l'UI directement

e0bc1e1
(⬆️ tout est là, à part la config next js de revert dans le dernier commit)

@UnbearableBear UnbearableBear requested a review from nkrmr April 7, 2020 14:54
@revolunet
Copy link
Member

pti prob de CI

@UnbearableBear
Copy link
Contributor Author

pti prob de CI

Elle était funky celle là mais j'ai trouvé le soucis !

@UnbearableBear
Copy link
Contributor Author

UnbearableBear commented Apr 7, 2020

Je vais essayer de virer le package que j'utilise pour le burger menu et en recoder un. C'est pas possible tout le code que ça importe pour faire un truc aussi bateau. Et rollup il l'aime pas mais alors pas du tout ce truc

@UnbearableBear UnbearableBear changed the title feat(ui): add burger nav to UI WIP - do not merge - do not review feat(ui): add burger nav to UI Apr 7, 2020
@UnbearableBear UnbearableBear changed the title WIP - do not merge - do not review feat(ui): add burger nav to UI WIP - do not merge - feat(ui): add burger nav to UI Apr 7, 2020
@UnbearableBear UnbearableBear changed the title WIP - do not merge - feat(ui): add burger nav to UI feat(ui): add burger nav to UI Apr 8, 2020
@UnbearableBear UnbearableBear requested a review from revolunet April 8, 2020 22:50
@UnbearableBear UnbearableBear merged commit 7f4e584 into master Apr 10, 2020
@UnbearableBear UnbearableBear deleted the manu/ui-burger branch April 10, 2020 09:25
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.

4 participants