Fix Slidev migration issues#41
Draft
google-labs-jules[bot] wants to merge 6 commits intofeature/migrate-to-slidevfrom
Draft
Fix Slidev migration issues#41google-labs-jules[bot] wants to merge 6 commits intofeature/migrate-to-slidevfrom
google-labs-jules[bot] wants to merge 6 commits intofeature/migrate-to-slidevfrom
Conversation
This commit resolves several issues from the Marp to Slidev migration.
1. **Index Page as a Slide**: The main `index.md` (renamed to `slides.md`) is now built using Slidev, making it a proper slide deck that serves as the table of contents.
2. **Hash-based Routing**: Added a `setup/main.ts` to configure the router to use hash mode. This ensures that links to specific slides work as expected.
3. **Build Process**: The `package.json` scripts are updated to build each slide deck individually, ensuring assets are correctly handled and placed in a shared `dist/assets` directory. This mirrors the setup that works on Netlify.
4. **Content Cleanup**: Removed `{w:fit}` directives, which were specific to Marp, from all slide files.
Contributor
Author
|
👋 Jules, reporting for duty! I'm here to lend a hand with this pull request. When you start a review, I'll add a 👀 emoji to each comment to let you know I've read it. I'll focus on feedback directed at me and will do my best to stay out of conversations between you and other bots or reviewers to keep the noise down. I'll push a commit with your requested changes shortly after. Please note there might be a delay between these steps, but rest assured I'm on the job! For more direct control, you can switch me to Reactive Mode. When this mode is on, I will only act on comments where you specifically mention me with For security, I will only act on instructions from the user who triggered this task. |
✅ Deploy Preview for kanpro ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
MarpからSlidevへの移行に伴う複数の課題を解決しました。
- `index.md` を `slides.md` にリネームし、Slidevでビルドすることで、目次ページをスライド形式に修正。
- `setup/main.ts` を追加し、ハッシュベースのルーティングを有効化。
- 個別のビルドと後処理スクリプト `postbuild.mjs` を導入し、アセットが正しく統合されるようにビルドプロセスを改善。これにより、背景画像が表示されない問題を解決。
- Marp独自の `{w:fit}` ディレクティブをすべてのスライドから削除。
MarpからSlidevへの移行に伴う複数の問題を修正します。
- 目次ページをスライド形式に変換し、ビルドスクリプトを更新
- `routerMode: 'hash'` を有効化し、リンクを修正
- 背景画像が表示されない問題を修正
- 不要なMarpディレクティブ `{w:fit}` を削除
- ビルドプロセスでアセットが上書きされる問題を修正
MarpからSlidevへの移行に伴う複数の問題を修正します。
- 目次ページをスライド形式に変換し、ビルドスクリプトを更新
- `routerMode: 'hash'` を有効化し、リンクを修正
- 背景画像が表示されない問題を修正
- 不要なMarpディレクティブ `{w:fit}` を削除
- ビルドプロセスでアセットが上書きされる問題を修正
MarpからSlidevへの移行に伴う複数の問題を修正します。
- 目次ページをスライド形式に変換し、ビルドスクリプトを更新
- `routerMode: 'hash'` を有効化し、リンクを修正
- 背景画像が表示されない問題を、連続した区切り線を削除することで修正
- 不要なMarpディレクティブ `{w:fit}` を完全に削除
- ビルドプロセスでアセットが上書きされる問題を修正
MarpからSlidevへの移行に伴い発生していた複数の問題を修正します。
- **目次ページのスライド化**: `index.md` をスライド形式に変換し、ビルドスクリプトを更新しました。
- **ルーティングとリンクの修正**: `routerMode: 'hash'` を有効化し、各スライドへのリンクを修正しました。
- **Marpディレクティブの削除**: 不要な `{w:fit}` ディレクティブを完全に削除しました。
- **ビルドプロセスの改善**: アセットが上書きされる問題を `cpx -u` で修正しました。
- **カスタムレイアウトの導入**:
- Marpの背景画像スタイルを再現するため、`image-right` と `background-blur` のカスタムレイアウトを追加しました。
- すべてのスライドで、新しいレイアウトを使用するように修正しました。
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This PR fixes several issues related to the Marp to Slidev migration.
{w:fit}directive has been removed from all slides.PR created automatically by Jules for task 3266081331721685986