Skip to content

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