Steven Engelbert

Results 142 comments of Steven Engelbert

> I think this PR introduced #777 ; can you please stylize these consistently? inconsistent font families makes the app feel unpolished. Could you point to an example of this...

> does this fix https://github.com/micro-nova/AmpliPi/issues/833 too? Let me deploy this and check real quick, if it does I'll connect that issue before merging so it gets closed as well

> Let me deploy this and check real quick, if it does I'll connect that issue before merging so it gets closed as well No

One failure with this present implementation is that it doesn't work on mobile, which makes it a bit of a nonstarter imo but it was my initial thought, I've found...

The easiest way to test this, if you want to see it work, is have the same unit open in two windows and close the source that the other window...

I've requested review from @linknum23 as well, as Klayton is out of office today Lincoln, it's alright if you don't have time for this PR, I've primarily requested your review...

This branch should include an "autoselect new source" before merging; That is, if you hit the add button and create a source it should be selected for you automatically

This issue is labeled `needs user input`, please :+1: react if you think this would be a good thing to spend our time on

![image](https://github.com/user-attachments/assets/eb7f7744-94ef-49cc-88c1-eee5282cb48f) The issue comes from this class, when I deactivate it it snaps back to normal

Seems it's from a mui package updating, the current state of `main` is uploaded to inhouse and doesn't have it, yet the current state of main on my computer does