aframevr / aframe-registry

[DISCONTINUED] Curated collection of community A-Frame components.
https://aframe.io/registry/
MIT License
77 stars 39 forks source link

Add aframe-spritesheet-component to registry #54

Closed OpherV closed 7 years ago

OpherV commented 7 years ago

Added my component: https://github.com/EkoLabs/aframe-spritesheet-component

ngokevin commented 7 years ago

Thanks for submitting!

@OpherV @feiss Do you think 2D spritesheets would be commonly useful in 3D/VR applications?

OpherV commented 7 years ago

Yes I do. After all, there is no other way to insert pre-baked animations into a 3d scene.

Common uses :

feiss commented 7 years ago

Yeah, this is nice and can be useful, thanks @OpherV !

I although would like to have a way of specify the current image in an absolute fashion. Now there's only a progress parameter from 0 to 1. I think the user should be able to specify also an actual index in the spritesheet mode or the key id in the json dictionary.

Great addition, anyway!

fernandojsg commented 7 years ago

Just out of curiosity, the sprite is aligned to the original camera position right? not to each one of the per-eye cameras used for stereo render

fernandojsg commented 7 years ago

Sorry my fault, I thought it was about a billboard component :)

OpherV commented 7 years ago

@feiss sure - I agree that API would be useful as well.

@ngokevin are you considering having this as part of core? If so I can finish implementing @feiss's suggestion and submit a PR

ngokevin commented 7 years ago

The Registry is a fine place for this component for now. Let me know when you get @feiss's suggestion down. Thanks!

ngokevin commented 7 years ago

Can you add a link to the examples in the Website section in GitHub? https://cloud.githubusercontent.com/assets/674727/24439774/2640bef6-1405-11e7-90d7-c39b4f876ce1.png

dmarcos commented 7 years ago

@OpherV we're looking forward to including this in the registry. It's almost ready to go. Do you have some time to bring it back home? Thank you!

OpherV commented 7 years ago

@dmarcos yeah! I'll sort things out this week :)

OpherV commented 7 years ago

@dmarcos

I've implemented @feiss's API suggestions

@ngokevin what do you mean by

Can you add a link to the examples in the Website section in GitHub?

Not sure what's this image you've linked 😶

ngokevin commented 7 years ago

Sorry for the delay, hope to get back to maintaining the Registry soon.