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

Wildcard selector for listener. #2279

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

HichemBenali
Copy link

Proposed Changes

  • Added wildcard to the listener allowing the usage of Listener with Matrix.

@tabuna
Copy link
Member

tabuna commented Jun 19, 2022

Hi @HichemBenali, But after all, this will only work for matrix rows that already existed, but were not added, due to the fact that the event is only registered when the page is loaded?

@HichemBenali
Copy link
Author

Hi @tabuna you are correct, I forgot that aspect. I believe this can be solved by either attaching a MutationObserver to the matrix table or by adding a custom event to the matrix it self then listening for that event.

@HichemBenali
Copy link
Author

HichemBenali commented Jun 21, 2022

@tabuna

Updated Proposal.

  • Added wildcard to the listener allowing the usage of Listener with Matrix.
  • Added MutationObserver to automatically add event listeners to newly added matrix fields.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants