Grey out any games that have an 'uninstalled' status

[jthoward64]

Big changes aside from obious UI stuff is the movement of where the status for the shown game is and the addition of a cached check for installed vs unistalled.

[dmadisetti]

I think this is a nice change. Thank you.

I will try it out this afternoon. Code looks good, my one comment is I don't think that the cache should be an App attribute, but I get that might be tricky with the borrow checker + loop

[jthoward64]

I think this is a nice change. Thank you.

I will try it out this afternoon. Code looks good, my one comment is I don't think that the cache should be an App attribute, but I get that might be tricky with the borrow checker + loop

Yeah, if you can think of a better place to put it I'd be glad to move it and update the PR, but I couldn't figure a good way

[dmadisetti]

Still at work, but thought about this some more. Can't we store status in the game object?

steam-tui/src/interface.rs

Line 128 in fe535c9

pub struct Game {

Then no need for cache

[jthoward64]

Still at work, but thought about this some more. Can't we store status in the game object?

steam-tui/src/interface.rs

Line 128 in fe535c9

pub struct Game {

Then no need for cache

Maybe, I was just trying to keep the scope of the changes small, but I'll give that a shot

[jthoward64]

Alright, list color will now be determined after the games are loaded in, this COULD cause issues, but I think it should work. The way I wrote it should also make it easy to add other install status colors (i.e. needs update) and probably more.

[jthoward64]

Still at work, but thought about this some more. Can't we store status in the game object?

steam-tui/src/interface.rs

Line 128 in fe535c9

pub struct Game {

Then no need for cache

Anything else I should take a look at here?

[dmadisetti]

Closed by 0.2.0 ? Let me know what you think