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

Update svelte-jsx.d.ts #408

Merged
merged 5 commits into from
Aug 2, 2020
Merged

Update svelte-jsx.d.ts #408

merged 5 commits into from
Aug 2, 2020

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?

packages/svelte2tsx/svelte-jsx.d.ts Outdated Show resolved Hide resolved
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