Skip to content

docs(kanvas): crop the black band from three tutorial screenshots - #1271

Merged
leecalcote merged 1 commit into
masterfrom
fix/tutorial-screenshot-bands
Sep 21, 2026
Merged

leecalcote merged 1 commit into
masterfrom
fix/tutorial-screenshot-bands

Conversation

@hiyach28

Copy link
Copy Markdown
Contributor

Fixes #1270

Three screenshots on the Kanvas tutorials still carry the flat black band that #1245 removed from the Sprint A captures. #1244 copied them from the audit set before that crop landed, so the tutorial copies were never cropped.

Image Page Before After
collaborative-editing/designer-comments.png collaborative-editing 1280x800, 93KB 1280x656, 80KB
collaborative-editing/designer-share.png collaborative-editing 1280x800, 102KB 1280x656, 89KB
publishing-to-catalog/designer-details-publish.png publishing-to-catalog 1280x800, 98KB 1280x656, 82KB

Verification

For each file:

Checked against the files served from production before the change, so this is the live state rather than just the repository. Filenames are unchanged, so no Markdown moves.

#1244 added three screenshots that are byte-identical to the original
Sprint A captures from #1245, copied before those captures were cropped.
Each was 1280x800 with rows 656 to 799 a flat #111111 across the full
width. Crop to 1280x656.

For each file the removed rows were verified to be all #111111 and every
kept pixel identical to the original. The result also matches the crops
already verified for the same captures in #1245.

Fixes #1270

Signed-off-by: hiyach28 <hiyach28@gmail.com>
@coderabbitai

coderabbitai Bot commented Sep 21, 2026

Copy link
Copy Markdown

Important

Review skipped

Review was skipped due to path filters

⛔ Files ignored due to path filters (3)
  • content/en/kanvas/tutorials/images/collaborative-editing/designer-comments.png is excluded by !**/*.png
  • content/en/kanvas/tutorials/images/collaborative-editing/designer-share.png is excluded by !**/*.png
  • content/en/kanvas/tutorials/images/publishing-to-catalog/designer-details-publish.png is excluded by !**/*.png

CodeRabbit blocks several paths by default. You can override this behavior by explicitly including those paths in the path filters. For example, including **/dist/** will override the default block on the dist directory, by removing the pattern from both the lists.

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Advanced

Run ID: 9b131ad0-3711-479a-b54a-6aff1c0781a0

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@github-actions

Copy link
Copy Markdown
Contributor

🚀 Preview deployment: https://docs.layer5.io/pr-preview/pr-1271/

Note: Preview may take a moment (GitHub Pages deployment in progress). Please wait and refresh. Track deployment here

@leecalcote
leecalcote merged commit 6058714 into master Sep 21, 2026
5 checks passed
@leecalcote
leecalcote deleted the fix/tutorial-screenshot-bands branch September 21, 2026 22:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Tutorial screenshots still carry the 144px black band removed in #1245

2 participants