Allow u16 index buffers

[Nazariglez]

closes #163

[jawadsh123]

looking nice 👌

a slightly tangential observation
with_data for all the three buffers takes different i/p

• for VertexBufferBuilder it takes &[f32]
• for IndexBufferBuilder it takes &[u32]
• for UniformBufferBuilder it takes BufferData

while set_buffer_data is buffer agnostic and always takes BufferData
wondering if there's value in unifying all to BufferData?

[Nazariglez]

We could. Though I am not sure if it is worth it, taking a trait generates worst error messages, I am not sure about if anyone will need to pass other types to the index buffer or the vertex either, and it will add some complexity to the code too. I guess it's possible, not a big fan of it right now if we don't have a strong use case.

What are your thoughts on it?

PD: Closing but we can keep talking about the types.

[jawadsh123]

don't have a strong case for it, was just wondering for consistency 😅
though now that I think about it more deeply, f32 for vertices and u32 for indices actually make total sense for the general use case
and for the odd outliers, set_buffer_data does it :)

[Nazariglez]

Yeah, methods that do the same with different types... is weird. I see the problem there. Though it may be fine for now. We can revisit this again at any time if you think we need to. Thanks!