Skip to content
This repository has been archived by the owner on Feb 16, 2022. It is now read-only.

Add delay to grid update #63

Open
denisinvader opened this issue Aug 1, 2017 · 2 comments
Open

Add delay to grid update #63

denisinvader opened this issue Aug 1, 2017 · 2 comments

Comments

@denisinvader
Copy link
Contributor

When grid is on and I change gutters size by holding arrow up (or down), grid toggles many times very fast and then disapears

@tonytopper
Copy link
Contributor

This could be tricky to implement. You don't want the interface to feel less responsive. I suggest timing it carefully biasing towards making sure the interface feels really responsive. Also, it is kind of nice to see the grid as it changes so I am not sure this is an actual problem, other than the flickering visual..

There is also an issue I've seem pop up where the grid is actually getting inserted twice in some cases. It might be good to look at that first, before trying to tackle this.

@denisinvader
Copy link
Contributor Author

I meant do not delay every time, but throttle / debounce toggle event handler (or rendering) function that will execute no more than once every 500 milliseconds (for example).

I agree that it's not very important bug, and twice inserted grid is more important.

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

No branches or pull requests

2 participants