Skip to content

Conversation

@CrosRoad95
Copy link

@CrosRoad95 CrosRoad95 commented Jan 6, 2019

Fixes Mantis #6064

Syntax:

  1. dxDrawPrimitive3D( primitiveType, postGui, vertex1, vertex2, ... )
    where vertex is {x,y,z} or {x,y,z,color}
  2. dxDrawMaterialPrimitive3D( primitiveType, texture, postGui, vertex1, vertex2, ... )
    where vertex is {x,y,z, u, v} or {x,y,z,color, u, v}

Test resource

TestResource

Example result:

image
image

Only alpha has issues when 2 rendering on top of each other 😢
https://github.com/multitheftauto/mtasa-blue/pull/760/files#diff-7950232204280a1f9935357a84b79876R80
if someone could help me fix it that would be nice, or we could do this in future

botder
botder previously requested changes Jan 12, 2019
Copy link
Member

@botder botder left a comment

Choose a reason for hiding this comment

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

Run the formatting tool after fixing these.

@botder botder changed the title feature: dxDrawPrimitive3D and dxDrawMaterialPrimitive3D Add dxDrawPrimitive3D and dxDrawMaterialPrimitive3D Jan 12, 2019
qaisjp
qaisjp previously requested changes Jan 12, 2019
@patrikjuvonen
Copy link
Contributor

PR is now up to date and I cleaned up requested changes confirming if they were done or not. @CrosRoad95 See my comments, I think there's still some changes that need doing.

Regarding function returns, this PR should wait for #821 to resolve.

@botder botder added this to the Backlog milestone Mar 4, 2019
@CrosRoad95
Copy link
Author

any update?

@qaisjp
Copy link
Contributor

qaisjp commented May 22, 2019

this PR should wait for #821 to resolve.

@botder
Copy link
Member

botder commented Aug 17, 2019

The decision in pull request #821 (Pin down return value on failure) is now settled.

@patrikjuvonen patrikjuvonen self-assigned this Sep 2, 2019
@patrikjuvonen patrikjuvonen removed the help wanted Extra attention is needed label Sep 2, 2019
@patrikjuvonen patrikjuvonen modified the milestones: Backlog, 1.6 Sep 2, 2019
@patrikjuvonen patrikjuvonen dismissed stale reviews from qaisjp and botder September 2, 2019 16:24

Resolved.

@patrikjuvonen patrikjuvonen merged commit 901ed44 into multitheftauto:master Sep 2, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants