-
-
Notifications
You must be signed in to change notification settings - Fork 744
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Normally you should be able to derive PrefabData for this using the macro. @Rhuagh ?
amethyst_utils/src/ortho_camera.rs
Outdated
@@ -86,6 +89,35 @@ impl Component for CameraOrtho { | |||
type Storage = DenseVecStorage<Self>; | |||
} | |||
|
|||
/// CameraOrtho prefab |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Would be nice to have a tiny bit more doc.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Heh, I'lll try the derive. Hadn't seen that.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do you think it still needs docs if it's just a derive?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Btw, can you add an entry to the changelog? Thanks!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You also need to add a link to your PR under the Fixed section of the changelog like this:
[#1188]: https://github.com/amethyst/amethyst/pull/1188
7c1f1d6
to
6fb32ea
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This looks good to me! I'd say let's merge this.
cc @amethyst/engine-devs
@mtsr Unfortunately there's a conflict with the changelog (yes, this happens too often, I'll fix it soon). Would you mind rebasing this PR so we can merge it? |
6fb32ea
to
753a742
Compare
Just rebased it. |
Thanks! 👍 bors r=azriel91,jojolepro |
Build succeeded |
I wanted to be able to set the CameraOrtho world coordinates in a Prefab, so I created a quick CameraOrthoPrefab.