OpenHands icon indicating copy to clipboard operation
OpenHands copied to clipboard

fix: fix multiple frontend warnings

Open Sparkier opened this issue 1 year ago • 2 comments

Sparkier avatar Apr 15 '24 22:04 Sparkier

@Sparkier the code LGTM, but can you describe a bit what problem you're solving here? Or is there an associated issue?

rbren avatar Apr 15 '24 23:04 rbren

  1. There is a console error since we have a button inside a button, hence the moving out of the refresh and collapse buttons.
  2. There is a warning because stopPropagation is default in React now.

Sparkier avatar Apr 16 '24 02:04 Sparkier

Oh nice, love that stopPropagation is default now, that always got me 😄

rbren avatar Apr 16 '24 14:04 rbren

Thanks for the fixes!

rbren avatar Apr 16 '24 14:04 rbren