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

Enhanced ImageをラップしたImage.svelteを作る #320

Closed
wants to merge 11 commits into from

Conversation

ryoppippi
Copy link
Member

ryoppippi.comの実装において、enhanced:imgをラップしたコンポーネントを用いると取り回しが良かったので採用する

@ryoppippi
Copy link
Member Author

ryoppippi commented Oct 5, 2024

まず、enhanced:imgをアプデ

@ryoppippi ryoppippi force-pushed the feature/Image.svelte branch from 8df5e3c to e93b614 Compare October 5, 2024 17:14
Copy link

cloudflare-workers-and-pages bot commented Oct 5, 2024

Deploying vim-jp-radio-lp with  Cloudflare Pages  Cloudflare Pages

Latest commit: d4431e2
Status: ✅  Deploy successful!
Preview URL: https://400a6b89.lp-d1f.pages.dev
Branch Preview URL: https://feature-image-svelte.lp-d1f.pages.dev

View logs

@ryoppippi
Copy link
Member Author

svelte自身をアプデ。エラーが消えることを確認

@ryoppippi ryoppippi force-pushed the feature/Image.svelte branch from 687e483 to b25e5c4 Compare October 5, 2024 17:20
@ryoppippi
Copy link
Member Author

enhanced:imgをラップしたImage.svelteを作成

@ryoppippi
Copy link
Member Author

各々のcomponentでImage.svelteを使用するよう変更

@ryoppippi
Copy link
Member Author

Uncaught (in promise) TypeError: s.getAttribute is not a function

これが最新のsvelteで発生してるので一旦様子見

@ryoppippi
Copy link
Member Author

#321
これ待ち

@ryoppippi ryoppippi force-pushed the feature/Image.svelte branch from 725935a to d4431e2 Compare October 5, 2024 19:27
@ryoppippi
Copy link
Member Author

rebaseしてforce pushしてしまったので、やり直しな機運

@ryoppippi
Copy link
Member Author

svelte/enhanced-img/sveltekitのアプデは別ぶらんちでやって良さそうなのでそっちに移動する

@ryoppippi
Copy link
Member Author

一旦closeしてPRし直す

@ryoppippi ryoppippi closed this Oct 7, 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