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

cross-border arrows do not zoom in #6580

Open
bernieEV opened this issue Mar 26, 2024 · 8 comments
Open

cross-border arrows do not zoom in #6580

bernieEV opened this issue Mar 26, 2024 · 8 comments

Comments

@bernieEV
Copy link

  1. start the URL https://app.electricitymaps.com/map
  2. klick on country AUSTRIA
  3. the map zoomed in, but the "cross-border arrow overlay" did not zoom in.
@VIKTORVAV99
Copy link
Member

Hi!

I can't seem to replicate the issue.

Could you provide some more information such as which browser or platform you are using the app on?

@VIKTORVAV99 VIKTORVAV99 added bug 🐞 frontend 🎨 needs triage Issues that need to be triaged and manually labeled. labels Mar 31, 2024
@phiphou
Copy link
Contributor

phiphou commented Apr 9, 2024

I have the same problem.

When I click on a zone, map zooms to the selected zone but arrows are not "synchronized".

If I move the map with arrow keys, arrows are still not updated.

But when I move the map with mouse, as soon I release the mouse, arrows updates and are now well placed.

I'm on Windows 11 and Chrome 123. I also encounter the same problem with Firefox 124 and Edge 123.

FireShot Capture 027 - Electricity Maps - Émissions CO₂ de la consommation électrique en tem_ - app electricitymaps com

@VIKTORVAV99
Copy link
Member

I just made some changes related to the arrows (released today), any indication it made things better?

I will also be making further changes to some internal logic that might affect this.

@phiphou
Copy link
Contributor

phiphou commented Apr 9, 2024

My tests have been made with the today's new version.

@VIKTORVAV99
Copy link
Member

My tests have been made with the today's new version.

Thanks for getting back to me, will see what I can do about it.

@VIKTORVAV99 VIKTORVAV99 self-assigned this Apr 9, 2024
@VIKTORVAV99 VIKTORVAV99 removed the needs triage Issues that need to be triaged and manually labeled. label Apr 9, 2024
@bernieEV
Copy link
Author

bernieEV commented Apr 9, 2024 via email

@phiphou
Copy link
Contributor

phiphou commented Apr 27, 2024

Any news about this quite anoying bug ?

@VIKTORVAV99
Copy link
Member

Any news about this quite anoying bug ?

Yeah we are planning on fixing it but there are some underlying fixes in #6651 we want to get merged first.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants