<d1b2>
<azonenberg> @fredzo_72653 @josHua so at a high level i do have a question wrt all of your sidebar work
<d1b2>
<azonenberg> Why use links for the timebase etc that opens the properties dialog
<d1b2>
<azonenberg> Rather than putting control widgets directly there and skipping a step
<d1b2>
<josHua> I think 1) because I think I remember that you expressed that you did not want that, and 2) you probably want to change multiple things at a time anyway
<d1b2>
<josHua> and in genreal I think the parameter-setting UI neesd much more thinking than it has gotten, so replicating the thing I don't love there is not great
<_whitenotifier>
[scopehal-apps] azonenberg 6349695 - StreamBrowserDialog: properly cache channel / global enable status
<d1b2>
<fredzo_72653> Oh yeah, thanks for that @azonenberg! Did it fix the 11-FPS issue ? If you find some time time to merge https://github.com/ngscopeclient/scopehal/pull/902, I would have seen it with a real PSU, I only tested with DemoPSU...
Stephie has joined #scopehal
Stephie- has quit [Ping timeout: 272 seconds]
<d1b2>
<azonenberg> yes 6349695 fixed the issue
<d1b2>
<azonenberg> channel enable state isnt cached in drivers
<d1b2>
<azonenberg> re 902, it's on my list to go through probably tomorrow
<d1b2>
<fredzo_72653> @azonenberg @josHua re our discussion on PSU switches, here is a PR for the combo-based toggle switch to prevent accidental channel activation: https://github.com/ngscopeclient/scopehal-apps/pull/782
<d1b2>
<macaba> For what it’s worth - I like the density of the config options in the sidebar, and don’t mind having a dialog open too much. If a middle ground is found with similar density, all good.
<d1b2>
<macaba> I imagine a potential option would be to keep the density of the sidebar similar, text only and so on, and to significantly expand the dialog to make it more like a soft front panel (in the aspect that it’s more guided with tool tips and min/max range indication)
<d1b2>
<macaba> Though what is shown in the previous gif for the demo PSU looks fine. Maybe if I was to hover over the voltage field, it would show “Min: 0V, max: 30V” or something.