Add Atlas Cropper documentation#113
Open
justkahdri wants to merge 12 commits intomainfrom
Open
Conversation
Contributor
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Draft of log 13 covering the atlas-cropper tool: what it does, its four-tab workflow, capabilities, and keyboard shortcuts. Includes a placeholder section for the Joyco use case. https://claude.ai/code/session_018ausLaQNYXDo46Tb2mzC5Q
443a26c to
f36269d
Compare
Comment on lines
+102
to
+106
| {/* ## Our use case | ||
|
|
||
| TODO: Add the specific Joyco use case here — what project drove the need, what the atlas workflow looks like in practice, before/after. | ||
|
|
||
| */} |
Contributor
There was a problem hiding this comment.
Uncommitted TODO left in published content
The commented-out ## Our use case section contains an explicit TODO marker and is still included in the file. Even though it's wrapped in an MDX comment block and won't render to users, it signals unfinished work checked into the main branch. Consider either removing the block entirely until the content is ready, or tracking it as a separate issue/task so it doesn't silently persist.
Prompt To Fix With AI
This is a comment left during a code review.
Path: content/logs/13-atlas-cropper.mdx
Line: 102-106
Comment:
**Uncommitted TODO left in published content**
The commented-out `## Our use case` section contains an explicit TODO marker and is still included in the file. Even though it's wrapped in an MDX comment block and won't render to users, it signals unfinished work checked into the main branch. Consider either removing the block entirely until the content is ready, or tracking it as a separate issue/task so it doesn't silently persist.
How can I resolve this? If you propose a fix, please make it concise.* fix: add 'use client' directive and remove unnecessary polling in team cookie hook Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com> * fix: redirect unauthorized access to toolbox UI based on team cookie presence * fix order --------- Co-authored-by: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
- Updated the positioning of scrollbar elements in the ScrollAreaViewport to ensure they align correctly with the edges of the viewport. - Simplified padding classes in ScrollAreaContent for better readability and maintainability.
- Introduced a new documentation file for "Containers" detailing all container variants and their differences. - Updated meta.json to include the new "containers" page and reorganized the existing pages for better structure.
- Replaced references from 'shiki' to 'mdx' in various components for better organization. - Updated 'shiki' package version from 3.15.0 to 3.23.0 in package.json and pnpm-lock.yaml. - Introduced new library handling features, including fetching library README files and slugs. - Added new library documentation files for 'Metri', 'Suno', and 'Susano'. - Enhanced sidebar and layout components to support library navigation.
- Added new hostname 'placehold.co' to the Next.js configuration for image handling. - Updated layout to include a badge for the new '/toolbox/atlas-cropper' route. - Improved documentation in 'atlas-cropper.mdx' to clarify output generation and added detailed use case examples with visual aids. - Updated 'meta.json' to include the 'atlas-cropper' page for better navigation.
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.
Summary
Added comprehensive documentation for the Atlas Cropper tool, a browser-based visual utility for defining, arranging, and extracting crop regions from atlas images.
Changes
content/logs/13-atlas-cropper.mdx) with complete feature overview and usage guideKey Sections
The documentation is structured to serve both as an introduction for new users and a reference guide for existing ones.
https://claude.ai/code/session_018ausLaQNYXDo46Tb2mzC5Q
Greptile Summary
Adds a new log entry documenting the Atlas Cropper browser tool, covering its four-tab workflow, keyboard shortcuts, JSON output format, and rationale. The content is well-written and consistent with the existing log format — no
meta.jsonis needed for this section as the logs directory uses auto-discovery (nocontent/logs/meta.jsonexists for any of the 12 prior entries).Confidence Score: 5/5
Safe to merge — only a static documentation file with no logic or security surface.
Single MDX file addition with no functional code changes. The one P2 finding (commented-out TODO) is cosmetic and doesn't affect rendering or functionality.
No files require special attention.
Vulnerabilities
No security concerns identified. This PR adds only static MDX documentation content with no code execution paths.
Important Files Changed
Prompt To Fix All With AI
Reviews (1): Last reviewed commit: "add atlas-cropper log entry draft" | Re-trigger Greptile