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

Funding page #318

Merged
merged 28 commits into from
Dec 19, 2024
Merged

Funding page #318

merged 28 commits into from
Dec 19, 2024

Conversation

dafuga
Copy link
Contributor

@dafuga dafuga commented Dec 18, 2024

No description provided.

@dafuga dafuga mentioned this pull request Dec 18, 2024
3 tasks
Copy link

cloudflare-workers-and-pages bot commented Dec 18, 2024

Deploying 2nicove with  Cloudflare Pages  Cloudflare Pages

Latest commit: 9d3fa4f
Status:⚡️  Build in progress...

View logs

@dafuga dafuga marked this pull request as ready for review December 18, 2024 16:34
Copy link
Contributor

@deansallinen deansallinen left a comment

Choose a reason for hiding this comment

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

Some changes, let me know if you'd rather I make a PR

Copy link
Contributor

Choose a reason for hiding this comment

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

Goes for all the images:

We should put these in the src/lib/assets directory, then import them in the <script> tag of the component. That way Vite can preprocess them. Also, in my branch for the homepage I'm adding the @svelte/enhanced-img plugin which will take it one step further and generate a srcset for each image when using <enhanced:img> tag.

We can upgrade to the enhanced image tag later, but for now I'd say let's put the images in a folder in assets

Copy link
Contributor

Choose a reason for hiding this comment

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

If you rebase on dev you should now be able to use <enhanced:img> for these assets

https://svelte.dev/docs/kit/images#sveltejs-enhanced-img

dafuga and others added 5 commits December 18, 2024 15:01
Co-authored-by: Dean Sallinen <[email protected]>
Co-authored-by: Dean Sallinen <[email protected]>
Co-authored-by: Dean Sallinen <[email protected]>
@aaroncox aaroncox marked this pull request as draft December 18, 2024 20:17
@dafuga dafuga marked this pull request as ready for review December 19, 2024 01:28
@aaroncox aaroncox merged commit d07c500 into dev Dec 19, 2024
2 of 3 checks passed
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.

3 participants