react-arborist icon indicating copy to clipboard operation
react-arborist copied to clipboard

Re-think handlers.select option.

Open jameskerr opened this issue 2 years ago • 1 comments

handlers.select(e, {selectOnClick: true})

That's weird.

jameskerr avatar Mar 18 '22 20:03 jameskerr

Should it be just (e) like other handlers?

I suppose by "weird" you mean these are possible:

onClick={(e) => select(e, { selectOnClick: false })
onHover={(e) => select(e, { selectOnClick: true })

thien-do avatar Oct 16 '22 17:10 thien-do