BalloonPanelView should not be focusable

[oleq]

Because of that we end up with problems like https://github.com/ckeditor/ckeditor5-link/issues/90. It's probably just a matter of using the @mixin ck-unselectable in SASS.

[Reinmar]

Isn't that CSS supported by Chrome only?

[oleq]

Nope. It's supported by almost every major web browser.

[Reinmar]

And why is the balloon focusable in the first place?

[oleq]

Most certainly because the author (me) has never considered this thing becoming a problem and obviously was wrong ;-)

[Reinmar]

There must've been some reason. I even recall there was something... But I don't remember what and whether this is still any relevant. Hopefully not... but I remember that CKE4's dialogs were also focusable.

[oleq]

but I remember that CKE4's dialogs were also focusable.

They aren't. Or at least, they don't look like they accept focus.

[Reinmar]

They aren't. Or at least, they don't look like they accept focus.

That's odd. I was nearly sure they were. So... I'll better stop confusing you :D

[oskarwrobel]

// Make this element `focusable` to be available for adding to FocusTracker.
tabindex: -1

https://github.com/ckeditor/ckeditor5-ui/blob/master/src/panel/balloon/balloonpanelview.js#L131

[oleq]

Maybe we don't need it anymore, hard to tell.