Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Suggestion: deprecate activeElement #100

Closed
ollema opened this issue Jun 24, 2024 · 2 comments
Closed

Suggestion: deprecate activeElement #100

ollema opened this issue Jun 24, 2024 · 2 comments
Labels
enhancement New feature or request

Comments

@ollema
Copy link

ollema commented Jun 24, 2024

Describe the feature in detail (code, mocks, or screenshots encouraged)

I suggest to deprecate and/or remove the activeElement utility and instead refer to the now builtin activeElement binding on <svelte:document>.

Alternatively, the activeElement could be rewritten to reuse the new builtin binding if you would like to keep it.

What type of pull request would this be?

Enhancement

Provide relevant links or additional information.

sveltejs/svelte#11879

@ollema ollema added the enhancement New feature or request label Jun 24, 2024
@TGlide
Copy link
Member

TGlide commented Jun 25, 2024

Runed is meant to work outside of Svelte components as well, so we'll keep the utility, but a note in the docs would be helpful.

@TGlide TGlide closed this as not planned Won't fix, can't repro, duplicate, stale Jun 25, 2024
@notramo
Copy link

notramo commented Dec 9, 2024

@TGlide, could you reopen this? The docs hasn't been updated yet.

Also worth mentioning the read-only focused binding which exists on every Element. It may be a cleaner solution for some use cases.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

3 participants