Viewport transformation efficiency

[white-axe]

Viewport transformation efficiency

[Speak2Erase]

I'm honestly not sure how to solve this one?

Viewports are currently projection matrices from model space (coordinates for vertices relative to each other) to clip space (position on the screen)

The way they're currently implemented, they actually don't do a lot for us, because egui calls RenderPass::set_viewport to set the region where custom paint callbacks are drawn on the screen.
This is both a curse and a blessing because it means that if we decide to draw things like events inside luminol_graphics::MapView we'll need to set the viewport ourselves, but it also means that we can effectively ditch Viewport (for events)

[white-axe]

We can still get rid of the projection matrix for Viewport though because, as far as I know, every single Viewport projection matrix we currently use is created using glam::Mat4::orthographic_rh, and that function just generates a matrix that represents some combination of translation and scaling (no rotations, shearing or other weird stuff). That can be reduced to just passing to the shader the translation as vec2<f32> and the scale as vec2<f32> instead of using a matrix.

[Speak2Erase]

I think what I'll do is split viewport into the region on the screen being drawn to (shared between things drawn in the samw region), and any transformations being applied to what's being drawn

This is how 3d usually works (there's a transformation matrix that applies rotation, scaling, and translation to an object) and a projection matrix which applies perspective and normalizes coordinates to fit in clip space