floating-columnState

[BSd3v]

columnState floats during grid interaction and only gets pushed when sent in a callback
columnDefs trumps columnState if it is pushed in a callback without a columnState
Markdown renderer now will pass back '' instead of undefined if there isnt a value

[alexcjohnson]

Is there some reason it doesn't work to just click the button? If it's scrolled off-screen maybe we can just put it on a new line or shorten the labels or something?

Suggested change

	(
	ActionChains(dash_duo.driver)
	.move_to_element(dash_duo.find_element('#load-column-defs'))
	.click()
	).perform()
	dash_duo.find_element('#load-column-defs').click()

[BSd3v]

Clicking it is too fast for some reason.

There has to be a slight pause between clicks or it doesn't work.

[alexcjohnson]

ok - maybe just add a little time.sleep then, with a comment about why?

[BSd3v]

I was trying to avoid that. Haha.

But yes, a time.sleep works.

[alexcjohnson]

haha yeah time.sleep is generally not great, but if there's a real reason we need a delay rather than waiting for a specific event, it's better to do that explicitly rather than something implicit like this.

[alexcjohnson]

💃 Looks great, just the last minor comment about time.sleep then we can merge. Very nice fixes and tests!