Skip to content

Provide utility to create shape for provided marker - #22977

Draft
linev wants to merge 4 commits into
root-project:masterfrom
linev:marker_shape
Draft

Provide utility to create shape for provided marker#22977
linev wants to merge 4 commits into
root-project:masterfrom
linev:marker_shape

Conversation

@linev

@linev linev commented Jul 31, 2026

Copy link
Copy Markdown
Member

Implement in central place functionality to create points which are necessary to paint marker.

Step by step use this functionality in different places like TGX11 or TGLPadPainter (there are 9 of such places)

@linev
linev requested review from bellenot and couet July 31, 2026 13:31
@linev linev self-assigned this Jul 31, 2026
@linev
linev requested review from dpiparo and pcanal as code owners July 31, 2026 13:31
@linev
linev marked this pull request as draft July 31, 2026 13:31
@linev
linev force-pushed the marker_shape branch 2 times, most recently from 87737ac to e662b27 Compare July 31, 2026 13:55
@github-actions

Copy link
Copy Markdown

Test Results

    23 files      23 suites   3d 14h 2m 43s ⏱️
 3 860 tests  3 806 ✅  0 💤 54 ❌
72 916 runs  72 830 ✅ 10 💤 76 ❌

For more details on these failures, see this check.

Results for commit e662b27.

@linev

linev commented Aug 3, 2026

Copy link
Copy Markdown
Member Author

@couet

It is still draft, I want to use introduced method in all other places - MacOS, GL, SVG, ...
And also not clear where the best place to put code.
For now I just add TAttMarker method, but one can put it somewhere else

linev added 4 commits August 3, 2026 09:45
Better format header and implementation files
For configured marker style and size returns
marker shape - dot, circle, fillcircle, polyline, fillarea, segments.

Such method should be used in all graphical classes where
marker should be painted
Replace same code,
direcly cast TPoint to XPoint
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