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

clusterCountLayer transition lags behind clusterLayer on cluster click #987

Open
maggievcody opened this issue Dec 22, 2021 · 5 comments
Open

Comments

@maggievcody
Copy link

maggievcody commented Dec 22, 2021

Description

When a cluster is clicked, the clusterCountLayer seems to lag behind the clusterLayer a little, resulting in the text transition being delayed. I notice this occurs in the cluster demo as well.

Adjusting the viewport transitionDuration doesn't seem to help – I think something may be off about the individual layer transition durations or order in which they occur in. It would be nice if these layers transitioned simultaneously so the cluster count is not delayed.

Screen Shot 2021-12-22 at 12 43 31 PM

Screen Shot 2021-12-22 at 12 44 26 PM

Screen Shot 2021-12-22 at 12 43 58 PM 1

Repro Steps

  • Click on cluster
  • See text in clusterCountLayer lag behind clusterLayer transition

Environment

  • Library Version: react-map-gl 6.1.14
  • React Version: 17.0.2
  • Browser Version: Chrome 96.0.4664.93
  • OS: Mac OS 11.2.3 Big Sur
@ezetaraborelli
Copy link

Hi @maggievcody! I know it's been more than two years from the day you posted this, but I'm having the same issue, and I am wondering if you found a solution for this? (and if you remember it)

@maggievcody
Copy link
Author

Hi @maggievcody! I know it's been more than two years from the day you posted this, but I'm having the same issue, and I am wondering if you found a solution for this? (and if you remember it)

Sorry, I don't work with this tool anymore and don't recall finding a solution for this.

@ezetaraborelli
Copy link

Oh bummer. Well, thank you for replying! Have a nice day

@m345054
Copy link

m345054 commented Jul 30, 2024

Hi @ezetaraborelli have you found a solution in the meantime?

@taraborellieze
Copy link

Hi @m345054, I'm sorry, but I have not.
I needed this for an interview test, which I failed (it wasn't this issue's fault), so I didn't have the time to try to fix it myself

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

No branches or pull requests

4 participants