search

Macchina-CLI / macchina

A system information frontend with an emphasis on performance.
https://crates.io/crates/macchina
MIT License
1.24k stars 47 forks source link

Implement palette.spacing feature #308

Closed feefs closed 2 months ago

feefs commented 3 months ago

See #307, this implements a new palette.spacing feature.

I'd like some extra help with review since I'm fairly new to Rust, and I'd like to adhere to the "emphasis on performance" tenet.

grtcdr commented 2 months ago

Looks good to me.

Not only does this make spacing much nicer to work with, but you've made it so the width of the palette is taken into consideration when the box is finally rendered, this fixes cases where the box can collide with the palette. Wonderful.