Skip to content

minor docs styling updates#48

Open
skarim wants to merge 2 commits intoskarim/fix-onto-rebasefrom
skarim/docs-styling
Open

minor docs styling updates#48
skarim wants to merge 2 commits intoskarim/fix-onto-rebasefrom
skarim/docs-styling

Conversation

@skarim
Copy link
Copy Markdown
Collaborator

@skarim skarim commented Apr 17, 2026

  • Add proper meta tags for social sharing previews
  • Fix stack diagram styling to invert colors properly in Safari dark mode

Copilot AI review requested due to automatic review settings April 17, 2026 04:43
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Updates the documentation landing page visuals and improves social-sharing metadata for the Astro/Starlight docs site.

Changes:

  • Replaces the static stack diagram SVG asset usage with an inline StackDiagram Astro component.
  • Adds Open Graph/Twitter meta tags (and updates the site description) for richer link previews.
  • Adds a social card image asset and removes the old stack-diagram.svg file.
Show a summary per file
File Description
docs/src/content/docs/index.mdx Swaps embedded diagram image markup for a reusable StackDiagram component.
docs/src/components/StackDiagram.astro Introduces an inline SVG diagram component with theme-aware styling.
docs/src/assets/stack-diagram.svg Removes the old static SVG asset no longer referenced.
docs/public/github-social-card.jpg Adds a social preview image referenced by OG/Twitter meta tags.
docs/astro.config.mjs Updates site description and adds OG/Twitter metadata tags (and removes robots noindex).

Copilot's findings

Tip

Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comments suppressed due to low confidence (1)

docs/src/components/StackDiagram.astro:61

  • The wrapper hard-codes text colors (#1f1f1f / #ffffff), but the docs theme already defines --sl-color-text (and switches it by data-theme). Using the theme variable here will keep the SVG aligned with future palette changes and avoid mismatches if additional themes are added.
<style>
  .stack-diagram-wrapper {
    color: #1f1f1f;
  }
  :root[data-theme='dark'] .stack-diagram-wrapper {
    color: #ffffff;
  }
  • Files reviewed: 3/5 changed files
  • Comments generated: 3

Comment thread docs/src/components/StackDiagram.astro
Comment thread docs/astro.config.mjs
Comment thread docs/astro.config.mjs
@skarim skarim force-pushed the skarim/fix-onto-rebase branch from 614f5bd to b4c2af0 Compare April 17, 2026 05:28
@skarim skarim force-pushed the skarim/docs-styling branch from e13e4d0 to 5ee094e Compare April 17, 2026 05:28
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.

2 participants