Skip to content

[Autocomplete][Doc] Fix event listeners are not removed #1227

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

Closed
wants to merge 1 commit into from

Conversation

smnandre
Copy link
Member

@smnandre smnandre commented Oct 23, 2023

Fix the documentation snippet, causing listeners to not be removed on disconnect.

From

this.element.removeEventListener('autocomplete:pre-connect', this._onConnect);
this.element.removeEventListener('autocomplete:connect', this._onPreConnect);

To

this.element.removeEventListener('autocomplete:connect', this._onConnect);
this.element.removeEventListener('autocomplete:pre-connect', this._onPreConnect);

@smnandre
Copy link
Member Author

smnandre commented Nov 8, 2023

Kind poke @weaverryan (as the online version could confuse users)

@weaverryan
Copy link
Member

Fixed in #1272 ... because I checked the PR's in the wrong order. Thanks Simon :)

@weaverryan weaverryan closed this Dec 12, 2023
weaverryan added a commit that referenced this pull request Dec 12, 2023
…doc examples (smnandre)

This PR was merged into the 2.x branch.

Discussion
----------

[Doc] Fix preConnect/connect inversion in "disconnect()" doc examples

Following #1227 i found other similar cases of callback inversion in the documentation.

Commits
-------

3f8218f Fix preConnect/connect inversion in "disconnect()" doc examples
symfony-splitter pushed a commit to symfony/ux-cropperjs that referenced this pull request Dec 12, 2023
…doc examples (smnandre)

This PR was merged into the 2.x branch.

Discussion
----------

[Doc] Fix preConnect/connect inversion in "disconnect()" doc examples

Following symfony/ux#1227 i found other similar cases of callback inversion in the documentation.

Commits
-------

3f8218f5 Fix preConnect/connect inversion in "disconnect()" doc examples
symfony-splitter pushed a commit to symfony/ux-swup that referenced this pull request Dec 12, 2023
…doc examples (smnandre)

This PR was merged into the 2.x branch.

Discussion
----------

[Doc] Fix preConnect/connect inversion in "disconnect()" doc examples

Following symfony/ux#1227 i found other similar cases of callback inversion in the documentation.

Commits
-------

3f8218f5 Fix preConnect/connect inversion in "disconnect()" doc examples
symfony-splitter pushed a commit to symfony/ux-typed that referenced this pull request Dec 12, 2023
…doc examples (smnandre)

This PR was merged into the 2.x branch.

Discussion
----------

[Doc] Fix preConnect/connect inversion in "disconnect()" doc examples

Following symfony/ux#1227 i found other similar cases of callback inversion in the documentation.

Commits
-------

3f8218f5 Fix preConnect/connect inversion in "disconnect()" doc examples
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants