Philip Jackson

I propagated the error using a boxed trait object to force things to the same type: ```rust |_| -> Box { println!("Wait ..."); let d = Duration::from_secs(3); match Timeout::new(d, &handle)...

Yes, that's perfect 👌 Will use that in the future Is there ever a case where you wouldn't want an already flattened future though? That API would seem more ergonomic...

The difference between minified and unminified is because React wraps event handlers inside other events in the development build to get better devtool integration. Problem is that Edge and IE...

I'm working on a PR, but thought I would check the approach I'm taking. Ideally event handlers should work the same way in dev and production. And React should retain...

Thanks @noahtallen those are good points. The `initSentry` is binding event listeners to `window`, so it really is more of an app-level concern. The reason I think it's worth trying...

I'm not quite sure about the cleanest way to do this. I'll just leave it for now. Next time a need comes up it might be more obvious what to...

@vykes-mac Saxon is AFK unfortunately. Let's remove the hover for now. I think it makes everything a bit complicated and I don't think having the repeated icons is that bad....

Hi @halleshubham, thanks for your interest in the new sites management page! This task in particular will be a bit tricky without access to Figma, which isn't accessible outside of...

Screenshots for reference:    

Changes look good to me. @SaxonF said that he had intentionally given the other empty states a different style, so I'd leave that up to him. I like the consistency,...