Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@volkyeth
Copy link
Owner

  • Add drawing order & layering guidelines section with visual example
  • Add head cropping guidelines section with example image
  • Add good trait design example section
  • Incorporate all 3 images that were originally added in PR Updated guidelines page #38
  • Apply changes to both Nouns and Lil Nouns variants
  • Restore visual enhancements that were lost during Next.js app router migration

- Add drawing order & layering guidelines section with visual example
- Add head cropping guidelines section with example image  
- Add good trait design example section
- Incorporate all 3 images that were originally added in PR #38
- Apply changes to both Nouns and Lil Nouns variants
- Restore visual enhancements that were lost during Next.js app router migration

Co-authored-by: volkyeth <[email protected]>
@vercel
Copy link

vercel bot commented Jun 10, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
lil-noundry ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jun 10, 2025 9:21pm
2 Skipped Deployments
Name Status Preview Comments Updated (UTC)
lil-studio ⬜️ Ignored (Inspect) Visit Preview Jun 10, 2025 9:21pm
noundry-assets ⬜️ Ignored (Inspect) Visit Preview Jun 10, 2025 9:21pm

@volkyeth
Copy link
Owner Author

@claude bring in the changes from #42 that seem to have similarly been lost during migration from pages router to app router

also go ahead and fix the linter errors involving unescaped entities:

@noundry/gallery:build: 80:21  Error: `'` can be escaped with `&apos;`, `&lsquo;`, `&#39;`, `&rsquo;`.  react/no-unescaped-entities

@claude
Copy link

claude bot commented Jun 10, 2025

Claude finished @volkyeth's task in 5m 5s —— View job


Task Completed (Partial)

I've successfully fixed the linter errors involving unescaped entities! ✅

What I Completed

  • Read current Guidelines.tsx files to understand current state
  • Research PR Updating the Guidelines page #42 to understand what changes need to be restored
  • Fix linter errors involving unescaped entities (' characters)
  • Commit changes

Changes Made

Fixed linter errors: Changed "Here's" to "Here's" on line 80 in both Guidelines.tsx files
Committed changes: Successfully pushed the linter fixes to the PR branch

Still Need Information

PR #42 Content: I couldn't find any information about PR #42 in the repository history or files. Could you provide more details about what specific content from PR #42 needs to be restored to the Guidelines pages?

The current Guidelines files already include the enhanced content from PR #38 (drawing order guidelines, head cropping guidelines, and good trait design examples with images). Let me know what additional content from PR #42 should be added.


- Fix linter error by escaping apostrophe in "Here's" -> "Here&apos;s"
- Apply fix to both Nouns and Lil Nouns variants

Co-authored-by: volkyeth <[email protected]>
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