pell icon indicating copy to clipboard operation
pell copied to clipboard

don't break tabIndex

Open sfusato opened this issue 6 years ago • 2 comments

I use Pell as part of a form. Using 'TAB' to navigate the form fields, I'm first taken through all the buttons I have initialized for my Pell's instance before arriving in the editor with the cursor.

title_input (press 'TAB' here) description_input (I expect the cursor to be in the editor and not looping through the icons)

Quick fix: add tabIndex=-1 to the icon's button elements

Currently I have this workaround after Pell is initialized:

document.querySelectorAll('.pell-button').forEach(button => button.setAttribute('tabIndex', "-1"))

but it would be cool to have this built-in.

sfusato avatar Mar 08 '19 13:03 sfusato

+1, I'm resorting to these sort of hack as well but it's not clean as I got many pell instances being dynamically created.

caroso1222 avatar May 30 '19 15:05 caroso1222

Thanks, slated for V2

jaredreich avatar Jun 02 '19 12:06 jaredreich