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

Fix source components rendering quirks #735

Merged

Conversation

wawuwo
Copy link
Contributor

@wawuwo wawuwo commented Jun 4, 2024

Hi!

This a follow-up to #723 and another portion of changes to components to make them look a bit prettier. This time components from "sources" groups were in work.

With this PR also comes a new type of drawing primitive — the so called "Polyline" which allows to draw compound lines consisting of several segments and make good-looking joining of them. I had to add be able to draw nice arrow heads without strange edges etc.

There is a lot of affected components and I was to lazy (shame on me!) to make before/after images this time, but I think a change in component's rendering is something self-describing and easy to see so lack of before/after images shouldn't be a problem.

This type of primitive allows to draw complex figures
which consist of multiple lines. In contrast to drawing just
multiple independent lines, when drawing a polyline QPainter can
join segments in pretty way.
@wawuwo
Copy link
Contributor Author

wawuwo commented Jun 4, 2024

Oops, turns out I've confused directions of a couple of arrows. Preparing a --fixup commit…

@ra3xdh ra3xdh added this to the 24.3.0 milestone Jun 5, 2024
@ra3xdh ra3xdh merged commit ac7b899 into ra3xdh:current Jun 5, 2024
7 checks passed
@ra3xdh
Copy link
Owner

ra3xdh commented Jun 5, 2024

Thanks for the contribution! Merged.

@wawuwo wawuwo deleted the fix-source-components-rendering-quirks branch June 5, 2024 09:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants