Skip to content

Fix silkscreen knockout text rendering#747

Open
rushabhcodes wants to merge 4 commits intotscircuit:mainfrom
rushabhcodes:fix/silkscreen-knockout-text
Open

Fix silkscreen knockout text rendering#747
rushabhcodes wants to merge 4 commits intotscircuit:mainfrom
rushabhcodes:fix/silkscreen-knockout-text

Conversation

@rushabhcodes
Copy link
Copy Markdown
Contributor

@rushabhcodes rushabhcodes commented Mar 28, 2026

Summary

This PR fixes silkscreen knockout text so it behaves like copper knockout text in the 3D viewer.

Previously, silkscreen knockout text could appear black instead of revealing the board background. The silkscreen texture path now performs a knockout cutout pass that removes the glyph strokes from the
silkscreen texture, so knockout text shows the board color underneath as expected.

Before

image

After

image

Copilot AI review requested due to automatic review settings March 28, 2026 18:18
@vercel
Copy link
Copy Markdown

vercel bot commented Mar 28, 2026

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

Project Deployment Actions Updated (UTC)
3d-viewer Ready Ready Preview, Comment Apr 2, 2026 9:03pm

Request Review

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copy link
Copy Markdown
Contributor

@seveibar seveibar left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

why is one of them called CUSTOM_PAD? (makes no sense?)

Also it looks wrong- the knockout padding clearly doesn't correspond with the text inside, the right and bottom padding are incorrect relative to the left and top padding

@rushabhcodes
Copy link
Copy Markdown
Contributor Author

@rushabhcodes
Copy link
Copy Markdown
Contributor Author

also it looks wrong- the knockout padding clearly doesn't correspond with the text inside, the right and bottom padding are incorrect relative to the left and top padding

ig this is a circuit-to-canvas issue, i will investigate more

@rushabhcodes
Copy link
Copy Markdown
Contributor Author

rushabhcodes commented Mar 28, 2026

updating circuit-to-canvas should fix it

@rushabhcodes
Copy link
Copy Markdown
Contributor Author

rushabhcodes commented Apr 2, 2026

@Abse2001
image

@rushabhcodes rushabhcodes requested a review from seveibar April 2, 2026 21:03
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.

3 participants