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

New enum docs. #3470

Merged
merged 2 commits into from Jan 27, 2015
Merged

New enum docs. #3470

merged 2 commits into from Jan 27, 2015

Conversation

Chaosus
Copy link
Contributor

@Chaosus Chaosus commented Jan 27, 2015

Docs for
-VertexElementUsage
-VertexElementFormat
-TextureFilter
-TextureAddressMode
-SurfaceFormat
Fix licenses for all of those enums.
SurfaceFormat_Legacy is renamed to SurfaceFormatLegacy and marked internal instead of public.

None = 0,
/// <summary>
/// Rotate 180 degrees around the Y axis before rendering.
Copy link
Member

Choose a reason for hiding this comment

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

Are you sure this is rotation? I think this is not rotation at all... it is literally flipping the sprite so the top and bottom are switched, but the left and right are not.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Flip sprite to 180 degrees around the Y axis before rendering.

It is better ? I'l change as you wish.,

Copy link
Member

Choose a reason for hiding this comment

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

Oh... I guess Y axis is the important bit here. It is trying to describe flipping as a rotation around a single axis. That seems confusing to me. Maybe a better description:

Render the sprite reversed along the X axis.

@KonajuGames ?

@tomspilman
Copy link
Member

Looks all good to me... i'll merge once it finishes building.

tomspilman added a commit that referenced this pull request Jan 27, 2015
@tomspilman tomspilman merged commit d2c6816 into MonoGame:develop Jan 27, 2015
@Chaosus Chaosus deleted the EnumComments branch January 27, 2015 18:29
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.

None yet

3 participants