Drop `@zestia/ember-auto-focus`

appuniversum / ember-appuniversum

Ember addon wrapping the appuniversum components.

https://appuniversum.github.io/ember-appuniversum

MIT License

14 stars 11 forks source link

Drop `@zestia/ember-auto-focus` #455

Closed Windvis closed 6 months ago

Windvis commented 9 months ago

The addon stopped publishing to npm and only publishes to Github packages. AFAIK there is no way for us to configure this, so app wouldn't have to add some config to an npmrc file.

Some options:

inline a custom auto-focus modifier
remove the auto-focus behaviour from the AuAccordion component. I'm not 100% sure it is needed for a11y reasons. None of the accessible accordion examples I found had this behavior.
use a different addon

Windvis commented 9 months ago

My personal preference is simply removing the auto-focus behavior, but it needs to be verified if it would have no accessibility implications first.

Windvis commented 7 months ago

I noticed some apps are also using it, so inlining and exposing it as a new au-autofocus modifier might make sense?