Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Board editor: Render pad & via texts with Fontobene #1206

Merged
merged 1 commit into from Aug 24, 2023

Conversation

ubruhin
Copy link
Member

@ubruhin ubruhin commented Aug 23, 2023

So far the texts within pads and vias (pad number resp. net name) were drawn with a TrueType font, but it turned out this rendering was a huge performance bottleneck, slowing down the whole board editor on large designs. Especially on Windows the board editor became basically unusable (depending on zoom level).

Now rendering these texts with a Fontobene stroke font which is much faster.

So far the texts within pads and vias (pad number resp. net name) were
drawn with a TrueType font, but it turned out this rendering was a
huge performance bottleneck, slowing down the whole board editor on
large designs. Now rendering them with a Fontobene stroke font which
is much faster.
@ubruhin ubruhin added this to the 1.0.0 milestone Aug 23, 2023
@ubruhin ubruhin self-assigned this Aug 23, 2023
@dbrgn
Copy link
Member

dbrgn commented Aug 23, 2023

rendering was a huge performance bottleneck

Really? 😂 🤦🏻

@ubruhin
Copy link
Member Author

ubruhin commented Aug 23, 2023

Really? 😂 🤦🏻

Yes, really 🙂 I never really experienced this problem on Linux, but on Windows it's a very serious issue. And now with this change, I realized it even slightly speeds up the board editor on Linux, pan/zoom feels completely instant now even with larger designs 💪

I don't know if there was some issue with our implementation of TrueType font rendering. Maybe there would be a solution to speed that up somehow, but replacing it by Fontobene was not a big deal and since it's a simple stroke font it's no surprise that performance is better...

@ubruhin ubruhin merged commit 74a609e into master Aug 24, 2023
8 checks passed
@ubruhin ubruhin deleted the fontobene-pad-texts branch August 24, 2023 12:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Development

Successfully merging this pull request may close these issues.

None yet

2 participants