docs(capture-screenshots): update conventions to dashboard.mergify.com + 2x DPR#12128
Merged
Conversation
Contributor
Merge Protections🟢 All 6 merge protections satisfied — ready to merge. Show 6 satisfied protections🟢 🤖 Continuous Integration
🟢 👀 Review Requirements
🟢 Enforce conventional commitMake sure that we follow https://www.conventionalcommits.org/en/v1.0.0/
🟢 🔎 Reviews
🟢 📕 PR description
🟢 🚦 Auto-queueWhen all merge protections are satisfied, this pull request will be queued automatically. |
There was a problem hiding this comment.
Pull request overview
Updates the public-safe capture-screenshots skill documentation to match the current dashboard host and to align screenshot conventions with the repo’s existing 2× (retina) image set, while pointing contributors to the internal Playwright capture tool for authenticated screenshots.
Changes:
- Replaces
app.mergify.comwithdashboard.mergify.comin the capture workflow and prerequisites. - Documents the 2× DPR (
deviceScaleFactor: 2) convention and references the internal Playwright capture tool for authenticated shots. - Refreshes the “Common dashboard URLs” section and notes the optional
?login=&repository=query context.
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
.claude/skills/capture-screenshots/SKILL.md |
Updates host references and adds guidance pointing to the internal Playwright capture tool, while keeping the interactive capture workflow documented. |
.claude/skills/capture-screenshots/references/conventions.md |
Adds 2× DPR convention guidance and refreshes the canonical dashboard URL/path guidance. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
jd
marked this pull request as ready for review
July 16, 2026 11:58
…m + 2x DPR Refresh the public-safe capture-screenshots skill: app.mergify.com -> dashboard.mergify.com throughout, add deviceScaleFactor: 2 to the viewport conventions, and point to the internal Playwright capture tool in Mergifyio/skills (no auth internals). Per review: frame the ?login&repository query context as optional (docs links omit it) and drop '/app' from the frontmatter description. Split out of #12127 (Test Insights screenshots) at Julien's request. Refs MRGFY-8060 Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com> Change-Id: I10022e0f76204f339e8392fa780c4f8fbbb025b7
jd
force-pushed
the
devs/jd/jd/capture-screenshots-conventions/update-conventions-dashboard-mergify-com-2x-dpr--10022e0f
branch
from
July 20, 2026 07:54
b66a481 to
938a94c
Compare
Member
Author
Revision history
|
JulianMaurin
approved these changes
Jul 20, 2026
sileht
approved these changes
Jul 20, 2026
Contributor
Merge Queue Status
This pull request spent 2 minutes 54 seconds in the queue, including 1 minute 57 seconds running CI. Required conditions to merge
|
53 tasks
mergify
Bot
deleted the
devs/jd/jd/capture-screenshots-conventions/update-conventions-dashboard-mergify-com-2x-dpr--10022e0f
branch
July 20, 2026 08:16
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.
Refresh the public-safe capture-screenshots skill: app.mergify.com -> dashboard.mergify.com throughout, add deviceScaleFactor: 2 to the viewport conventions, and point to the internal Playwright capture tool in Mergifyio/skills (no auth internals).
Per review: frame the ?login&repository query context as optional (docs links omit it) and drop '/app' from the frontmatter description.
Split out of #12127 (Test Insights screenshots) at Julien's request.
Refs MRGFY-8060
Co-Authored-By: Claude Opus 4.8 (1M context) noreply@anthropic.com