Skip to content

docs(browser-extensions): reshoot the install screenshot - #12694

Open
jd wants to merge 1 commit into
mainfrom
devs/jd/jd/reshoot-extensions-list/reshoot-install-screenshot--f8367fb4
Open

docs(browser-extensions): reshoot the install screenshot#12694
jd wants to merge 1 commit into
mainfrom
devs/jd/jd/reshoot-extensions-list/reshoot-install-screenshot--f8367fb4

Conversation

@jd

@jd jd commented Sep 5, 2026

Copy link
Copy Markdown
Member

The install screenshot on the Browser Extensions page framed a dashboard
sidebar that no longer exists: top-level Integrations, Config Editor,
Commands Restrictions and Event Logs entries. Those moved into the Settings
sidebar — Integrations under Developer, Config Editor and Commands
Restrictions under Configuration — so the image contradicted the step above
it, which already tells you to open Settings → Integrations and pick the
Browser category.

Only the image was stale, so this is a reshoot with no text change. The alt
text already described the Browser category and the two cards, and it still
does.

The replacement frames the content pane only — the category rail with Browser
selected, plus the Chrome Extension and Firefox Add-On cards — instead of the
whole viewport. That follows the capture conventions, which say to crop the
sidebar out unless the navigation is the point, for exactly the reason this
image rotted: a shot built around the sidebar goes stale every time a nav item
moves, even when the feature it documents has not changed. It also drops the
org name, the signed-in user's name and their avatar that the old shot carried.

Captured headless at 1440x900, retina 2x, light theme, from the mergify-sandbox
org.

Fixes Mergifyio/ci-bot#389
MRGFY-8060

Co-Authored-By: Claude Opus 5 (1M context) noreply@anthropic.com
Claude-Session: https://claude.ai/code/session_01FREXr8FPsik88qcghADHcE

The install screenshot on the Browser Extensions page framed a dashboard
sidebar that no longer exists: top-level Integrations, Config Editor,
Commands Restrictions and Event Logs entries. Those moved into the Settings
sidebar — Integrations under Developer, Config Editor and Commands
Restrictions under Configuration — so the image contradicted the step above
it, which already tells you to open Settings → Integrations and pick the
Browser category.

Only the image was stale, so this is a reshoot with no text change. The alt
text already described the Browser category and the two cards, and it still
does.

The replacement frames the content pane only — the category rail with Browser
selected, plus the Chrome Extension and Firefox Add-On cards — instead of the
whole viewport. That follows the capture conventions, which say to crop the
sidebar out unless the navigation is the point, for exactly the reason this
image rotted: a shot built around the sidebar goes stale every time a nav item
moves, even when the feature it documents has not changed. It also drops the
org name, the signed-in user's name and their avatar that the old shot carried.

Captured headless at 1440x900, retina 2x, light theme, from the mergify-sandbox
org.

Fixes Mergifyio/ci-bot#389
MRGFY-8060

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01FREXr8FPsik88qcghADHcE
Change-Id: If8367fb480011d7bb31b92025a5e68d2c599f648
Copilot AI lite review requested due to automatic review settings September 5, 2026 06:26
@mergify
mergify Bot had a problem deploying to Mergify Merge Protections September 5, 2026 06:26 Failure

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.

Copilot wasn't able to review any files in this pull request.


💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@mergify

mergify Bot commented Sep 5, 2026

Copy link
Copy Markdown
Contributor

Merge Protections

🔴 2 of 6 protections blocking · waiting on 👀 reviews

Protection Waiting on
🔴 👀 Review Requirements 👀 reviews
🔴 🔎 Reviews 👀 reviews
🟢 🤖 Continuous Integration
🟢 Enforce conventional commit
🟢 📕 PR description
🟢 🚦 Auto-queue

🔴 👀 Review Requirements

Waiting for

  • #approved-reviews-by >= 2
This rule is failing.
  • any of:
    • #approved-reviews-by >= 2
    • author = dependabot[bot]
    • author = renovate[bot]
    • all of:
      • author = mergify-ci-bot
      • -head ~= ^docs-agent/

🔴 🔎 Reviews

Waiting for

  • #review-requested = 0
This rule is failing.
  • #review-requested = 0
  • #changes-requested-reviews-by = 0
  • #review-threads-unresolved = 0

Show 4 satisfied protections

🟢 🤖 Continuous Integration

  • 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/

🟢 Enforce conventional commit

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

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

🟢 📕 PR description

  • body ~= (?ms:.{48,})

🟢 🚦 Auto-queue

When all merge protections are satisfied, this pull request will be queued automatically.

@mergify
mergify Bot requested a review from a team September 5, 2026 06:28
@jd
jd marked this pull request as ready for review September 5, 2026 18:55
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.

2 participants