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.

Sign up for GitHub

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

Jump to bottom

Add support for usePassive option to the DomEventObserver #16412

Closed
Mgsy opened this issue May 23, 2024 · 0 comments · Fixed by #17098
Closed

Add support for usePassive option to the DomEventObserver #16412

Mgsy opened this issue May 23, 2024 · 0 comments · Fixed by #17098
Labels
squad:core Issue to be handled by the Core team. support:2 An issue reported by a commercially licensed client. type:feature This issue reports a feature request (an idea for a new functionality or a missing option).
Milestone
iteration 78 (v43...

Comments

@Mgsy
Copy link
Member

Mgsy commented May 23, 2024

📝 Provide a description of the new feature

Currently, DomEventObserver doesn't pass the usePassive option to the listener -https://github.com/ckeditor/ckeditor5/blob/5248c7be58ba444cc060d7b4ce7047376e0ccd1d/packages/ckeditor5-engine/src/view/observer/domeventobserver.ts#L73-L78. 

This feature request is about adding support for usePassive option in the DomEventObserver.

If you'd like to see this feature implemented, add a 👍 reaction to this post.
@Mgsy Mgsy added type:feature This issue reports a feature request (an idea for a new functionality or a missing option). support:2 An issue reported by a commercially licensed client. squad:core Issue to be handled by the Core team. labels May 23, 2024
timblack1-WC added a commit to timblack1-WC/ckeditor5 that referenced this issue Jun 3, 2024
@timblack1-WC
feat(domeventobserver): ckeditor#16412 add support for usePassive opt…
3fd861c 
…ion to the DomEventObserver

Fixes ckeditor#16412
@timblack1-WC timblack1-WC mentioned this issue Jun 3, 2024
Closed
@Mati365 Mati365 mentioned this issue Sep 16, 2024
Merged
@CKEditorBot CKEditorBot added this to the iteration 78 milestone Sep 17, 2024
@CKEditorBot CKEditorBot mentioned this issue Sep 18, 2024
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
squad:core Issue to be handled by the Core team. support:2 An issue reported by a commercially licensed client. type:feature This issue reports a feature request (an idea for a new functionality or a missing option).
Projects
None yet
Milestone
iteration 78 (v43.2.0)
2 participants
@Mgsy @CKEditorBot