Skip to content

Blog Post 2 - #2

Merged
OrigamiKing3612 merged 6 commits into
masterfrom
blog-post-002
Mar 27, 2026
Merged

OrigamiKing3612 merged 6 commits into
masterfrom
blog-post-002

Conversation

@OrigamiKing3612

Copy link
Copy Markdown
Owner

No description provided.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Adds a second blog post and improves blog/MDX rendering with per-post theming and richer code block styling (including Expressive Code integration).

Changes:

  • Add a new MDX blog post and wire blog cards/pages to use per-post color metadata.
  • Introduce markdown-specific styling (inline code, blockquotes, list markers) and extend theme background variables.
  • Upgrade Astro + MDX-related deps and add astro-expressive-code integration for enhanced code blocks.

Reviewed changes

Copilot reviewed 18 out of 19 changed files in this pull request and generated 3 comments.

Show a summary per file
File Description
src/styles/markdown.css New base-layer styles for rendered markdown content (code, blockquotes, markers, expressive-code wrapper).
src/styles/global.css Adds additional --background-* tokens for use by markdown/code styling.
src/pages/blog/index.astro Uses each post’s data.color when rendering the Card list.
src/pages/blog/[slug].astro Imports markdown styles, applies per-post CSS color var, passes description to layout, adjusts blog article styling, and removes custom MDX component mapping.
src/content/blog/002.beforeinstallprompt.mdx Adds “Blog Post 2” content.
src/components/content/ContentUl.astro Forwards props to <ul> (MDX wrapper behavior change).
src/components/content/ContentP.astro Forwards props to <p> (MDX wrapper behavior change).
src/components/content/ContentLi.astro Forwards props to <li> (MDX wrapper behavior change).
src/components/content/ContentH1.astro Forwards props and tweaks nested code styling.
src/components/content/ContentH2.astro Forwards props and tweaks nested code styling.
src/components/content/ContentH3.astro Forwards props and tweaks nested code styling.
src/components/content/ContentH4.astro Forwards props and tweaks nested code styling.
src/components/content/ContentH5.astro Forwards props and tweaks nested code styling.
src/components/content/ContentH6.astro Forwards props and tweaks nested code styling.
src/components/content/ContentCode.astro New wrapper component for <code> (currently not wired into rendering).
src/components/ContentComponents.astro Adds code mapping to the MDX components map (but map appears unused after page changes).
package.json Bumps Astro/MDX/RSS/Sitemap versions and adds astro-expressive-code.
astro.config.mjs Configures Expressive Code themes/selectors and adds markdown Shiki config.
pnpm-lock.yaml Lockfile updates for dependency bumps and new integration.
Files not reviewed (1)
  • pnpm-lock.yaml: Language not supported
Comments suppressed due to low confidence (1)

src/components/ContentComponents.astro:18

  • After removing the import/usage from the blog post page, this components mapping doesn’t appear to be referenced anywhere in src/ (search for ContentComponents/components={ returns no matches). Either wire this back into MDX rendering (e.g., pass it to Content) or delete the file/components to avoid carrying unused code.
export const components = {
    h1: ContentH1,
    h2: ContentH2,
    h3: ContentH3,
    h4: ContentH4,

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

Comment thread src/pages/blog/[slug].astro Outdated
Comment thread src/pages/blog/[slug].astro
Comment thread src/content/blog/002.beforeinstallprompt.mdx Outdated
@OrigamiKing3612
OrigamiKing3612 merged commit b20b4ba into master Mar 27, 2026
1 check passed
@OrigamiKing3612
OrigamiKing3612 deleted the blog-post-002 branch March 27, 2026 16:53
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