Joe Clark

Results 167 comments of Joe Clark

I very strongly suspect that the missing functions just need to declare `@public` EDIT June 10: No, at least some missing functions are public

Hey @christad92 I think we should bump priority on this and get it fixed. It's something that ought to be on my plate, maybe next week

Mailchimp shows no documentation at all!

implementation notes: * don't autolayout on edge creation * add a toolbar on the chart with zoom and layout controls * cross fingers on dagre

I'm fairly sure that edge conditions are valid and don't need to be represented as nodes. But yeah, debugging them is quite hard. Most of our debugging experience is pretty...

Yeah @midigofrank I can see that actually it's totally not working ¯\_(ツ)_/¯ I'm trying to dig into the root of the problem. This should just work really. I have a...

So, two issues I'm staring down. No progress on either 1) The main code editor flickers weirdly on resize. That's because for some reason some of the resize calls cause...

Ok, maybe it's a Friday afternoon, but this seems to be working. Suspiciously well actually. I'm gonna update the PR description.

I've plugged this in with [rc-resize-observer](https://www.npmjs.com/package/rc-resize-observer) but I wonder if [https://www.npmjs.com/package/@react-hook/resize-observer](https://www.npmjs.com/package/@react-hook/resize-observer) is a better bet (it uses one global observer so its more efficient). I'm gonna leave it for now...

@midigofrank we should be able to forward the refs through easily enough. I'll make an example for you. Not sure if I'll do it here or create a merge branch......