Skip to content

Fix global objectLimit behavior in InteractiveGraphics#117

Open
musakartalengin wants to merge 1 commit into
tscircuit:mainfrom
musakartalengin:fix-interactive-graphics-object-limit
Open

Fix global objectLimit behavior in InteractiveGraphics#117
musakartalengin wants to merge 1 commit into
tscircuit:mainfrom
musakartalengin:fix-interactive-graphics-object-limit

Conversation

@musakartalengin
Copy link
Copy Markdown

Summary

  • apply objectLimit globally across all filtered graphics objects instead of separately per object type
  • keep the limit warning based on the pre-limit filtered object count
  • normalize non-integer limits before slicing

Closes #42

Testing

  • npx biome format --write site\components\InteractiveGraphics\InteractiveGraphics.tsx
  • npm run build

@vercel
Copy link
Copy Markdown

vercel Bot commented May 10, 2026

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

Project Deployment Actions Updated (UTC)
graphics-debug Ready Ready Preview, Comment May 10, 2026 10:16pm

Request Review

@musakartalengin
Copy link
Copy Markdown
Author

Thanks! CI and the Vercel preview are passing. This should fix #42 by applying objectLimit globally after filtering, rather than per object type.

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.

Add option objectLimit that limits the number of objects displayed (after filters) on InteractiveGraphics

1 participant