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

Add antialiased parameter to canvas_item_add_triangle_array #10645

Closed
wants to merge 1 commit into from
Closed

Add antialiased parameter to canvas_item_add_triangle_array #10645

wants to merge 1 commit into from

Conversation

Geequlim
Copy link
Contributor

@Geequlim Geequlim commented Aug 26, 2017

The canvas_item_add_triangle_array use polygon item to draw but it is antilaiased as default so I add this option to make it is possiable to draw triangle arrays without antilaiased.

Bugsquad edit: Fixes #10461.

@Geequlim
Copy link
Contributor Author

Intel HD4600
A Spine animation drawn with antilaiased triangles:
image

And the animation drawn without antilaiased triangles:
image

@toger5
Copy link
Contributor

toger5 commented Aug 26, 2017

duplicate of: #10523

are the other things you do. necessary? It also works without but maybe they are still needed?

@Geequlim
Copy link
Contributor Author

@toger5 I think things are necessary to build on windows.

@toger5
Copy link
Contributor

toger5 commented Aug 26, 2017

okay so we merge this one than...
Since you also use false as the default value it fixes the style box outline too!

@bruvzg
Copy link
Member

bruvzg commented Aug 26, 2017

Is there any reason to have AA option for triangle array, current AA method (drawing outline with AA lines) is designed for polygons only and will not work for triangle arrays anyway.
Maybe just use polygon->antialiased = false;

@akien-mga akien-mga added this to the 3.0 milestone Aug 26, 2017
@akien-mga akien-mga changed the title Add antilaiased paramerter to canvas_item_add_triangle_array Add antialiased parameter to canvas_item_add_triangle_array Aug 26, 2017
@mrcdk
Copy link
Contributor

mrcdk commented Aug 27, 2017

This PR seems to also fix the ugly editor border issues #10461

@toger5
Copy link
Contributor

toger5 commented Aug 27, 2017

It definitely does!

akien-mga added a commit that referenced this pull request Aug 28, 2017
@akien-mga
Copy link
Member

Superseded by 9450179, @reduz confirmed that @bruvzg's suggestion was sufficient.

@akien-mga akien-mga closed this Aug 28, 2017
@Geequlim Geequlim deleted the pr-triangle_array-with-antialiased-option branch January 11, 2018 03:03
Xrayez added a commit to goostengine/goost that referenced this pull request Nov 22, 2020
Doesn't work well for closed outlines where drawing is done
with triangle array in Godot 3.2, see discussion in:

godotengine/godot#10645

Seems to work better for line drawing.
Xrayez added a commit to goostengine/goost that referenced this pull request Nov 22, 2020
Doesn't work well for closed outlines where drawing is done
with triangle array in Godot 3.2, see discussion in:

godotengine/godot#10645

Seems to work better for line drawing.
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.

5 participants