Skip to content

Conversation

@vielle-dev
Copy link

  • there was an extra L in the name
  • i cloned the project to have a nosey and it annoyed me lol

- there was an extra L in the name
- i cloned the project to have a nosey and it annoyed me lol
on attribute refers to the event which is triggered (click, blur, etc)
target attribute refers to the elements which will recive the event listener (css selector: #button, button.primary, a, etc)

children of the <event-> component are triggered whenever the event listener fires and on first page load (the later is an unintentional side effect which idk how to fix lol) (its late and idk web components well)
@vielle-dev
Copy link
Author

vielle-dev commented Nov 24, 2024

okay full clarity im still getting to grips w git commiting when theres like multiple branches n stuff i have no idea why its added the stuff in here and how to fix it orz
ill drop the <event-> commit message here and if u can like. merge parts of a big ol commit do that ig?
(also: ik the event component is probably not great and also has no test coverage if u want ill go and try make em tommorow :3)

Add <event-> component

on attribute refers to the event which is triggered (click, blur, etc)
target attribute refers to the elements which will recive the event listener (css selector: #button, button.primary, a, etc)

children of the component are triggered whenever the event listener fires and on first page load (the later is an unintentional side effect which idk how to fix lol) (its late and idk web components well)

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.

1 participant