Skip to content
This repository has been archived by the owner on Apr 2, 2024. It is now read-only.

Energy tooltip #330

Merged
merged 19 commits into from
Dec 9, 2022
Merged

Energy tooltip #330

merged 19 commits into from
Dec 9, 2022

Conversation

enzo-mourany
Copy link
Contributor

No description provided.

@enzo-mourany enzo-mourany added UI User interface related rc required Required for release candidate labels Nov 28, 2022
@enzo-mourany enzo-mourany changed the title WIP : Adding energy tooltip Adding energy tooltip Dec 2, 2022
@celian-rib
Copy link
Contributor

@enzo-mourany je sais pas si tu as terminé, si oui pense a nous mettre en reviewer qu'on recoive la notif ;)

@enzo-mourany enzo-mourany requested review from celian-rib and gfroidcourt and removed request for celian-rib and gfroidcourt December 3, 2022 22:41
@celian-rib celian-rib linked an issue Dec 4, 2022 that may be closed by this pull request
2 tasks
Copy link
Contributor

@celian-rib celian-rib left a comment

Choose a reason for hiding this comment

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

  • La barre n'est plus centré par rapport aux boutons :

image

  • La popup doit aussi etre clicable et cliquer dessus doit la faire disparaitre

src/components/effect/AnimatedFlask.js Outdated Show resolved Hide resolved
src/components/map/EnergyTooltip.js Outdated Show resolved Hide resolved
src/components/map/EnergyTooltip.js Outdated Show resolved Hide resolved
src/components/map/EnergyTooltip.js Show resolved Hide resolved
src/components/map/EnergyTooltip.js Show resolved Hide resolved
src/components/map/DropyMap.js Outdated Show resolved Hide resolved
@celian-rib
Copy link
Contributor

@enzo-mourany si tu as terminé, pense a redemander une review que j'ai une notif, et a fix les conflits ;)
image

@celian-rib celian-rib changed the title Adding energy tooltip Energy tooltip Dec 8, 2022
@celian-rib
Copy link
Contributor

@enzo-mourany conflit

Co-authored-by: Guillaume  <gfroidcourt@users.noreply.github.com>
@enzo-mourany enzo-mourany merged commit 77c570b into main Dec 9, 2022
@celian-rib celian-rib deleted the DF_279_Energy-tutorial-popup branch December 9, 2022 22:26
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
rc required Required for release candidate UI User interface related
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Energy tutorial popup
2 participants