TilemapChunk single quad; TileData (color/visibility)

[ConnerPetzold]

Objective

• Use a single quad to render a TilemapChunk
• Add support for tile color and visibility

Testing

• Tested using example -- there doesn't appear to be any visual tile bleeding or rounding issues. Open to ideas on further testing

[github-actions]

It looks like your PR has been selected for a highlight in the next release blog post, but you didn't provide a release note.

Please review the instructions for writing release notes, then expand or revise the content in the release notes directory to showcase your changes.

[copygirl]

Since this type is repr(C) and it's being transmuted to be sent to the GPU (from my understanding), could RGBA be stored in 4 separate bytes? Or is there something about endianness or something else I'm missing?

[ConnerPetzold]

Oh yeah good point -- maybe just do color: [u8; 4]?

[ConnerPetzold]

Oh right, I remember why I wrote it this way to begin with, although it's not super clear.. I wanted to make it more obvious that this was mapping to a Rgba16Uint texture, so each of these fields is one of the color channels. I guess the same can be accomplished with comments though. I'd love a clearer way to accomplish this.

[ConnerPetzold]

Maybe something like this?

#[repr(C)]
#[derive(Clone, Copy, Debug, Pod, Zeroable)]
pub struct PackedTileData {
    tileset_index: u16, // red channel
    color: [u8; 4], // green and blue channels
    flags: u16, // alpha channel
}

[copygirl]

Okay, admittedly that took me a while to understand. With the doc comment on the type it's definitely clearer.