phlask-map icon indicating copy to clipboard operation
phlask-map copied to clipboard

Pins blink on toolbar icon clicks on desktop and mobile

Open ravimakhijapi opened this issue 1 year ago • 3 comments

When one clicks some of the Toolbar icons (e.g. Add Site, Search, or the Filter), the map pins blink briefly. On mobile they only blink on the first click, but then will blink again if clicking a different toolbar and coming back to the first one.

The bug seems to have appeared in commit a489447d8f3468eaf6787bdff4aca0d904bd818e.

Expected behavior

There should be no blinking of pins when toolbar icons are clicked.

ravimakhijapi avatar Jun 06 '24 21:06 ravimakhijapi

Hey @ravicodelabs can I get assigned ?

hunxjunedo avatar Jun 07 '24 14:06 hunxjunedo

Hey @hunxjunedo, I'm actually not sure what the process for that would be here, and another team member may be able to help with that. cc: @tomporvaz.

However, feel free to fork and debug there and leave any thoughts/findings here for discussion if applicable.

Thank you for your interest!

ravimakhijapi avatar Jun 07 '24 14:06 ravimakhijapi

I think @RNR1 and I were discussing blinking issues in the last meeting as well - this might be fixed by our work on the markers, so I'll add a comment this weekend when I'll be working on my PR 👍

vontell avatar Jun 07 '24 16:06 vontell

Fixed by https://github.com/phlask/phlask-map/pull/554

patrickcbrady avatar Jan 22 '25 00:01 patrickcbrady