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 (most) typescript errors + use Vite #90

Draft
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

sakgoyal
Copy link

@sakgoyal sakgoyal commented Aug 15, 2024

do you want to use Vite or should I revert these changes and go back?

this has a major bug and does not work. I am unable to change any settings and they are not visible.

"vite": "^5.4.0"
},
"dependencies": {
"@dhdbstjr98/gif.js": "^1.0.2"
Copy link
Author

Choose a reason for hiding this comment

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

removes the need to manually download the gif.js files (gif.js, gif.map, gif.worker.js...)

serviceworker.ts Outdated
Copy link
Author

Choose a reason for hiding this comment

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

Vite has built in hot reloading

*/
// interface HTMLElement { class : string }

export type Tag<T extends HTMLElement = HTMLElement> = T & {
Copy link
Author

Choose a reason for hiding this comment

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

use generic functions to fix the massive any problem. This solution is almost perfect but there are issues with some errors in some places

@sakgoyal sakgoyal mentioned this pull request Aug 15, 2024
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