docs: integrate Scion explainer slideshow directly into landing page#287
Merged
Merged
Conversation
Replace the iframe-embedded slide deck with a fully inline, interactive slideshow component. All 7 slides from the hosted deck are now rendered directly in the landing page with scoped CSS and inline JavaScript. Key changes: - Full slideshow with keyboard/touch/click navigation, progress dots, and animated transitions - All interactive widgets recreated inline: State Model Simulator, Collaborators Graph (with dynamic spawn/delete loop), Reactive Notification Wakeup flow, and Shared Filesystem Simulator - Replace "Boot" with "Run" throughout all slide content - Remove Overview Deck iframe section; add "technical deep dive" link to the Google Slides deck below the inline slideshow - Move "How it works" quickstart section below "See it in Action" video - Scope all slideshow CSS under .scion-deck to avoid style conflicts with the landing page
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
Contributor
There was a problem hiding this comment.
Code Review
This pull request updates the project log documentation to reflect the integration of an interactive inline slideshow component into the docs-site landing page, replacing the previous iframe embed approach. The feedback suggests minor formatting and grammatical improvements in the markdown file, specifically wrapping the command curl in backticks and adding the article "the" before "file structure" for better readability.
| - The slides were fetched via curl since the WebFetch tool had model availability issues. | ||
| - Node.js 22+ was required for the Astro build, which wasn't available in the sandbox. Validated the file structure (frontmatter syntax, balanced HTML tags) programmatically instead. | ||
| - The landing page is a standalone Astro page (not a Starlight content doc), so it uses raw HTML/CSS rather than MDX components. | ||
| - The slides HTML/CSS/JS were fetched via curl (WebFetch tool had model availability issues) and carefully adapted for inline embedding. |
Contributor
There was a problem hiding this comment.
For better readability and consistency with standard markdown formatting, consider wrapping the command-line tool curl in backticks.
Suggested change
| - The slides HTML/CSS/JS were fetched via curl (WebFetch tool had model availability issues) and carefully adapted for inline embedding. | |
| - The slides HTML/CSS/JS were fetched via `curl` (WebFetch tool had model availability issues) and carefully adapted for inline embedding. |
| - Node.js 22+ was required for the Astro build, which wasn't available in the sandbox. Validated the file structure (frontmatter syntax, balanced HTML tags) programmatically instead. | ||
| - The landing page is a standalone Astro page (not a Starlight content doc), so it uses raw HTML/CSS rather than MDX components. | ||
| - The slides HTML/CSS/JS were fetched via curl (WebFetch tool had model availability issues) and carefully adapted for inline embedding. | ||
| - Node.js 22+ was required for the Astro build, which wasn't available in the sandbox. Validated file structure (balanced HTML tags, frontmatter syntax) programmatically. |
Contributor
There was a problem hiding this comment.
Adding the article 'the' before 'file structure' improves the grammatical flow of the sentence.
Suggested change
| - Node.js 22+ was required for the Astro build, which wasn't available in the sandbox. Validated file structure (balanced HTML tags, frontmatter syntax) programmatically. | |
| - Node.js 22+ was required for the Astro build, which wasn't available in the sandbox. Validated the file structure (balanced HTML tags, frontmatter syntax) programmatically. |
krohnjw
pushed a commit
to ParkWhiz/scion
that referenced
this pull request
Jun 3, 2026
…oogleCloudPlatform#287) * Integrate Scion explainer slideshow directly into landing page Replace the iframe-embedded slide deck with a fully inline, interactive slideshow component. All 7 slides from the hosted deck are now rendered directly in the landing page with scoped CSS and inline JavaScript. Key changes: - Full slideshow with keyboard/touch/click navigation, progress dots, and animated transitions - All interactive widgets recreated inline: State Model Simulator, Collaborators Graph (with dynamic spawn/delete loop), Reactive Notification Wakeup flow, and Shared Filesystem Simulator - Replace "Boot" with "Run" throughout all slide content - Remove Overview Deck iframe section; add "technical deep dive" link to the Google Slides deck below the inline slideshow - Move "How it works" quickstart section below "See it in Action" video - Scope all slideshow CSS under .scion-deck to avoid style conflicts with the landing page * Update project log for inline slideshow integration
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
1 participant
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.
Summary
Test plan