Skip to content

Add Atlas Cropper documentation#113

Open
justkahdri wants to merge 7 commits into
mainfrom
claude/draft-atlas-cropper-docs-jsWSv
Open

Add Atlas Cropper documentation#113
justkahdri wants to merge 7 commits into
mainfrom
claude/draft-atlas-cropper-docs-jsWSv

Conversation

@justkahdri

@justkahdri justkahdri commented Apr 8, 2026

Copy link
Copy Markdown
Contributor

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

  • Created new documentation file (content/logs/13-atlas-cropper.mdx) with complete feature overview and usage guide
  • Documented the four-tab workflow: Crop Definition, Layout Arrangement, Output, and UV Crop
  • Included keyboard shortcuts and quality-of-life features reference table
  • Provided JSON output format example
  • Explained the tool's purpose and differentiation from general-purpose design tools

Key Sections

  • Problem statement — addresses the pain point of manual atlas cropping workflows
  • Workflow documentation — detailed breakdown of each of the four sequential tabs with feature lists
  • Keyboard shortcuts — comprehensive reference table for power users
  • Output format — example JSON structure for crop configurations
  • Rationale — explains why this tool is better suited than Photoshop/Figma for atlas workflows
  • Links — reference to GitHub repository

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

This PR adds a new documentation page for the Atlas Cropper tool to the toolbox section, along with the necessary sidebar registration, badge update, and supporting image assets.

  • content/toolbox/atlas-cropper.mdx — Complete documentation covering the four-tab workflow (Crop Definition, Layout Arrangement, Output, UV Crop), keyboard shortcuts, JSON output format, and an end-to-end real-world use case with screenshots and video.
  • app/(registry)/layout.tsx / content/toolbox/meta.json — Registers the new page in the sidebar and moves the new badge from /toolbox/skills to /toolbox/atlas-cropper.
  • next.config.ts — Adds placehold.co to remotePatterns, but that domain is not used anywhere; likely a development leftover. Two uploaded image assets (uv-crop-3.png, uv-crop-4.png) are also unreferenced by the MDX file.

Confidence Score: 5/5

Safe to merge — changes are purely additive documentation and minor config; no logic or data paths are affected

All changes are new static content, sidebar wiring, and image assets. The only loose ends are an unused placehold.co remote pattern and two unreferenced PNG files, neither of which impacts rendering or runtime behaviour.

next.config.ts (unused remotePattern entry) and the two unreferenced image assets in public/atlas-cropper-entry/

Important Files Changed

Filename Overview
content/toolbox/atlas-cropper.mdx New documentation page for Atlas Cropper — well-structured with four-tab workflow breakdown, keyboard shortcut table, JSON output example, and a real-world use case walkthrough with screenshots and video
next.config.ts Adds placehold.co to remotePatterns — domain is not referenced anywhere in the codebase and appears to be a development artifact
app/(registry)/layout.tsx Moves the 'new' badge from /toolbox/skills to /toolbox/atlas-cropper as expected for the new entry
content/toolbox/meta.json Adds atlas-cropper as the first entry in the toolbox pages list
public/atlas-cropper-entry/uv-crop-3.png Added along with uv-crop-4.png but neither is referenced in the MDX documentation — likely leftover assets
Prompt To Fix All With AI
Fix the following 2 code review issues. Work through them one at a time, proposing concise fixes.

---

### Issue 1 of 2
next.config.ts:14-17
`placehold.co` is whitelisted as a remote image hostname but is not referenced anywhere in the codebase — the new MDX file uses only local `/public` images. This appears to be a development artifact left in after placeholder images were replaced with real screenshots. Leaving an unreferenced external domain in `remotePatterns` is unnecessary noise and makes the allowlist harder to audit.

### Issue 2 of 2
public/atlas-cropper-entry/uv-crop-3.png:1
**Unreferenced image assets**

`uv-crop-3.png` and `uv-crop-4.png` are added to `public/atlas-cropper-entry/` but are not referenced anywhere in `content/toolbox/atlas-cropper.mdx`. The documentation only uses `UV-crop.png`, `UV-crop-2.png`, `crop-definition.png`, and `layout-arrangement.png`. These two files appear to be either extras from an earlier draft or screenshots intended for a section that was removed before publishing.

Reviews (2): Last reviewed commit: "feat: add "Try it" section to atlas-crop..." | Re-trigger Greptile

@vercel

vercel Bot commented Apr 8, 2026

Copy link
Copy Markdown
Contributor

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
joyco-hub Ready Ready Preview, ✅ 1 resolved May 16, 2026 0:21am

Comment thread content/logs/13-atlas-cropper.mdx Outdated
claude and others added 4 commits May 15, 2026 16:55
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
- 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.
@justkahdri justkahdri self-assigned this May 15, 2026
@justkahdri

Copy link
Copy Markdown
Contributor Author

@greptile review

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants