Re-focusing the trigger, when closing via `esc` key about floating-vue

[Akryum]

What would be the use case for an additional parameter to hide? Doesn't it work if you just listen to hide and focus your button?

[dobromir-hristov]

That will focus if you click outside, or hide via some other mechanism, which is probably not correct 🤔 not exactly sure what is more correct, from an accessibility standpoint.

[Akryum]

Maybe you can do something similar to:

function onHide () {
  // Nothing is currently focused
  if (document.activeElement === document.body) {
    myButton.value.focus()
  }
}

[dobromir-hristov]

I'll see if always focusing on hide is a terrible idea. Will give feedback later.

[dobromir-hristov]

Right, that did the trick :)

  const el = document.activeElement
  const popperClass = 'v-popper__popper';
// if no active element, or its the body, or its the popper itself OR is still inside the popper
  if (!el || el === document.body || el.classList.contains(popperClass) || el.closest(`.${popperClass}`)) {
    // focus the trigger button
  }