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

fix: wrong custom event name for specific elements targeted events #408

Open
wants to merge 1 commit into
base: next
Choose a base branch
from

Conversation

gabrielalmeida
Copy link

Related Issue

No previous issue, but as it's clearly a mistake, PR was created.

Your solution

aos:out:specific-id custom events created for specific elements events handling via data-aos-id attributes were wrongly named as aos:in:specific-id instead of aos:out:specific-id, just renamed it accordingly.

`Out` custom events created for specific elements events handling via
`data-aos-id` were named as `aos:in:specific-id` instead of
`aos:out:specific-id`
@rlamana
Copy link

rlamana commented Dec 9, 2018

+1
@michalsnik Please review and merge.

@marcelaerts
Copy link

This is not fixed yet?

@burnified
Copy link

Fix this please!

@b-sharpe
Copy link

Would love to see this committed

@shikkaba
Copy link

shikkaba commented Nov 5, 2019

Was this fixed? I'm thinking no as this wasn't updated since last year.

@jinman-nuvo
Copy link

Is there a way we can promote or vote on this so it gets reviewed and merged? I'd rather not attach an EventListener to each and every animation.

@philwolstenholme
Copy link

This appears to still be an issue in 3.0.0-beta.6

@philwolstenholme
Copy link

For now I'm using https://www.npmjs.com/package/patch-package to patch the issue

@florek41
Copy link

Main package not fixed yet, but fix works well!

jankocian referenced this pull request in jankocian/aos Mar 25, 2021
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.

None yet

10 participants