cropped images for weave (top-level) - #3135
Open
anastasiaguspan wants to merge 5 commits into
Open
Conversation
Removes the Weave nav chrome from 128 screenshots under weave/. One image is flagged for human review. Originals preserved as <name>_scOLD.<ext>; HTML review reports in images/cropper-reports_scOLD/ (named weave-root_* to avoid colliding with the images/weave reports). Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Contributor
|
Preview deployment for your docs. Learn more about Mintlify Previews.
No preview available (4)
Updated 2026-08-27 15:54:59 UTC for deployment · 7c97c56. |
Contributor
|
Images automagically compressed by Calibre's image-actions ✨ Compression reduced images by 67%, saving 37.3 MB. Displaying top 25 most improved images. 199 images processed.
57 images did not require optimisation. |
Contributor
|
Images automagically compressed by Calibre's image-actions ✨ Compression reduced images by 11.3%, saving 149.4 KB.
252 images did not require optimisation. |
dbrian57
approved these changes
Aug 27, 2026
dbrian57
left a comment
dbrian57
left a comment
Contributor
There was a problem hiding this comment.
I had to replace two screenshots, but everything else here looked fine.
Contributor
|
Images automagically compressed by Calibre's image-actions ✨ Compression reduced images by 79.7%, saving 160.3 KB.
255 images did not require optimisation. |
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
2 participants
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.
Removes the Weave app chrome (left nav rail, and top bar where present) from 128 screenshots under the top-level
weave/tree — the largest folder of this series. One image (guides/evaluation/img/evals_tab.png) is flagged for human review.Each original is preserved next to its cropped version as
<name>_scOLD.<ext>, which powers the review reports below and GitHub's own image diff. Reports are namedweave-root_*to avoid colliding with the earlierimages/weavereports.Review
Rendered before/after reports (via raw.githack):
Tip: the Files changed tab also shows GitHub's native before/after viewer (2-up / swipe / onion skin) for every modified PNG.
🤖 Generated with Claude Code