Skip to content
This repository was archived by the owner on Jan 5, 2024. It is now read-only.

CF403 Added line with thickness - #415

Merged
garethyr merged 9 commits into
developmentfrom
CF403-Add-thickness-parameter
Oct 28, 2022
Merged

garethyr merged 9 commits into
developmentfrom
CF403-Add-thickness-parameter

Conversation

@Pzkane

@Pzkane Pzkane commented Feb 27, 2022

Copy link
Copy Markdown

Resolves #403
demo

@Pzkane
Pzkane requested review from MaximDude and garethyr February 27, 2022 00:48

@garethyr garethyr left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Thanks for working on this. I've left some comments on things that need changing.

Comment thread System/GraphicalPrimitive.cpp Resolved
Comment thread System/GraphicalPrimitive.cpp Resolved
Comment thread System/GraphicalPrimitive.cpp Resolved
Comment thread System/GraphicalPrimitive.cpp Resolved
Pzkane and others added 6 commits March 9, 2022 14:07
…s - much simpler and produces better result

Proposed solution starts leaving gaps between individual lines when drawing at an angle (above 10 or so degrees) and is less performant due to significantly more primitive draw calls (less complex ones, but still significantly more)

@garethyr garethyr left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

A few small changes

Comment thread System/GraphicalPrimitive.h Resolved
Comment thread CHANGELOG.md Resolved
Comment thread Managers/PrimitiveMan.cpp Resolved
@garethyr
garethyr merged commit 1473594 into development Oct 28, 2022
@garethyr
garethyr deleted the CF403-Add-thickness-parameter branch October 28, 2022 20:19
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add thickness parameter to PrimitiveMan:DrawLinePrimitive()

3 participants