Skip to content

Conversation

@jd
Copy link
Member

@jd jd commented Dec 3, 2025

This adds 2 buttons to copy for LLM and view as Markdown

Copilot AI review requested due to automatic review settings December 3, 2025 16:20
@mergify mergify bot had a problem deploying to Mergify Merge Protections December 3, 2025 16:20 Failure
@mergify
Copy link
Contributor

mergify bot commented Dec 3, 2025

Merge Protections

Your pull request matches the following merge protections and will not be merged until they are valid.

🟢 🤖 Continuous Integration

Wonderful, this rule succeeded.
  • all of:
    • check-success = build
    • check-success = lint
    • check-success = test
    • any of:
      • check-success = test-broken-links
      • label = ignore-broken-links
    • any of:
      • check-success=Cloudflare Pages
      • -head-repo-full-name~=^Mergifyio/

🟢 👀 Review Requirements

Wonderful, this rule succeeded.
  • any of:
    • #approved-reviews-by >= 2
    • author = dependabot[bot]
    • author = mergify-ci-bot

🟢 Enforce conventional commit

Wonderful, this rule succeeded.

Make sure that we follow https://www.conventionalcommits.org/en/v1.0.0/

  • title ~= ^(fix|feat|docs|style|refactor|perf|test|build|ci|chore|revert)(?:\(.+\))?:

🟢 🔎 Reviews

Wonderful, this rule succeeded.
  • #changes-requested-reviews-by = 0
  • #review-requested = 0
  • #review-threads-unresolved = 0

🟢 📕 PR description

Wonderful, this rule succeeded.
  • body ~= (?ms:.{48,})

@mergify mergify bot requested a review from a team December 3, 2025 16:24
Copilot finished reviewing on behalf of jd December 3, 2025 16:25
Copy link

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

This PR adds two new toolbar buttons to documentation pages: "Copy for LLM" (copies markdown source to clipboard for use with AI assistants) and "View as Markdown" (opens the raw markdown file in a new tab). The buttons are integrated into the page heading area with responsive layout support.

Key Changes

  • New toolbar buttons with compact size styling for documentation pages
  • Copy functionality with fallback from markdown source to rendered content
  • Responsive grid layout for breadcrumbs and toolbar actions

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 9 comments.

File Description
src/layouts/MainLayout.astro Restructures page heading to add toolbar buttons alongside breadcrumbs with responsive grid layout
src/components/CopyForLLMButton.astro Implements copy button with markdown fetching, fallback to rendered content, and state management
src/components/ViewMarkdownButton.astro Adds link button that opens markdown source in new tab
src/components/PageContent/PageContent.astro Adds CSS for page heading bar layout and responsive toolbar positioning

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

@mergify mergify bot had a problem deploying to Mergify Merge Protections December 3, 2025 16:32 Failure
@jd jd force-pushed the devs/jd/copy-for-llm/I59fbd6cb70f0c4d4c409b4f25ebcd587211abdf7 branch from 83bb30e to e7f0b44 Compare December 3, 2025 16:42
@mergify mergify bot had a problem deploying to Mergify Merge Protections December 3, 2025 16:42 Failure
@jd jd force-pushed the devs/jd/copy-for-llm/I59fbd6cb70f0c4d4c409b4f25ebcd587211abdf7 branch from e7f0b44 to c3cce56 Compare December 3, 2025 17:06
@mergify mergify bot had a problem deploying to Mergify Merge Protections December 3, 2025 17:06 Failure
@jd jd force-pushed the devs/jd/copy-for-llm/I59fbd6cb70f0c4d4c409b4f25ebcd587211abdf7 branch from c3cce56 to 16f1583 Compare December 3, 2025 17:10
@mergify mergify bot had a problem deploying to Mergify Merge Protections December 3, 2025 17:10 Failure
This adds 2 buttons to copy for LLM and view as Markdown

Change-Id: I59fbd6cb70f0c4d4c409b4f25ebcd587211abdf7
@jd jd force-pushed the devs/jd/copy-for-llm/I59fbd6cb70f0c4d4c409b4f25ebcd587211abdf7 branch from 16f1583 to bd8cf22 Compare December 3, 2025 17:12
@mergify mergify bot deployed to Mergify Merge Protections December 3, 2025 17:12 Active
@mergify
Copy link
Contributor

mergify bot commented Dec 4, 2025

Merge Queue Status Beta

✅ The pull request has been merged

This pull request spent 1 minute 54 seconds in the queue, including 1 minute 34 seconds running CI.
The checks were run on draft #9567.

Required conditions to merge

mergify bot added a commit that referenced this pull request Dec 4, 2025
@mergify mergify bot added the queued label Dec 4, 2025
@mergify mergify bot merged commit e52f5c9 into main Dec 4, 2025
8 checks passed
@mergify mergify bot deleted the devs/jd/copy-for-llm/I59fbd6cb70f0c4d4c409b4f25ebcd587211abdf7 branch December 4, 2025 08:10
@mergify mergify bot removed the queued label Dec 4, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants