Skip to content

Conversation

@BlackFenix2
Copy link
Contributor

Adding missing type definitions for on:intro and on:outro start and end events. this project is using the frontendmasters course from https://github.com/Rich-Harris/cameoparison-starter

Type '{ onoutrostart: () => boolean; onoutroend: () => boolean; class: string; }' is not assignable to type 'HTMLProps<HTMLDivElement>'.
  Property 'onoutrostart' does not exist on type 'HTMLProps<HTMLDivElement>'.ts(2322)

Adding missing type definitions for on:intro and on:outro start and end events.

```
Type '{ onoutrostart: () => boolean; onoutroend: () => boolean; class: string; }' is not assignable to type 'HTMLProps<HTMLDivElement>'.
  Property 'onoutrostart' does not exist on type 'HTMLProps<HTMLDivElement>'.ts(2322)
```
Copy link
Member

@jasonlyu123 jasonlyu123 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks! I have some suggestions for the typing issue. Can you take a look?

Copy link
Contributor Author

@BlackFenix2 BlackFenix2 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I made the changes (bit noobish with file review)

@jasonlyu123
Copy link
Member

you have some linting error with trailing spaces you can run yarn lint --fix to fix it

@BlackFenix2
Copy link
Contributor Author

just ran the linter, i thought i could get away with editing the file on github directly 😊

Let me know if you need anything else.

@jasonlyu123
Copy link
Member

Great! thanks

@jasonlyu123 jasonlyu123 merged commit ebf6149 into sveltejs:master Aug 2, 2020
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