stuerp / foo_vis_spectrum_analyzer

A foobar2000 component that displays a spectrum analyzer.
MIT License
20 stars 0 forks source link

Selecting another graph on the graph list does not update some options #34

Closed WagnerGFX closed 8 months ago

WagnerGFX commented 8 months ago

Where?

How it's working now:

boxes1 boxes2

Expected Behavior:

System Info:

Notes:

stuerp commented 8 months ago

Thank you for the detailed report. Confirmed. I don't know how I missed that. Fixed in the next version.

WagnerGFX commented 8 months ago

When testing the alpha1 version, I can confirm the Graph settings are holding when changing between each graph item, at least for the drop boxes, check boxes and number fields. The ones still not behaving well are the arrow buttons for the number fields, they don't update when I change the values manually or switch between graphs.

One example is changing the amplitude range from -70 to -90 manually. When I press the arrow down after that, it will change to -71 instead of -91. The same still applies when changing between graphs.

The gradient color % seem to always apply spread whenever I change, add or remove a color.

stuerp commented 8 months ago

Good catch. The up/down controls are fixed in the next version.

The gradient color % was addressed in 0.7.4.1.

stuerp commented 8 months ago

@WagnerGFX ,

Included in: https://github.com/stuerp/foo_vis_spectrum_analyzer/releases/tag/v0.7.5.0-alpha1

WagnerGFX commented 8 months ago

Good catch. The up/down controls are fixed in the next version.

Switching graphs update the values correctly, the arrows also follow that value. On the other hand, I still get an issue where I change a value manually and the arrows still consider the previous value when clicked.

The gradient color % was addressed in 0.7.4.1.

The gradient % now stay in place when adding or removing colors, but I'm not being able to edit them by double-clicking anymore. Also, the % doesn't update when changing between visual elements. I have to click again on the selected color to update the box.

I've also noticed that, when Amplitude-Based is enabled, changing color sources for Bar Area has no effect and it keeps applying the gradient color. If needed, I can open a separate issue, as it seems to be a side effect of the new amplitude feature.

stuerp commented 8 months ago

I've also noticed that, when Amplitude-Based is enabled, changing color sources for Bar Area has no effect and it keeps applying the gradient color. If needed, I can open a separate issue, as it seems to be a side effect of the new amplitude feature.

That's to be expected, isn't it? The amplitude determines the color. How did you expect it to behave?

WagnerGFX commented 8 months ago

That's to be expected, isn't it? The amplitude determines the color. How did you expect it to behave?

Ops, I didn't explain it correctly 😅

Right now, when the Amplitude checkbox is enabled, changing to another color source will still keep gradient applied. I have to move back to gradient, disable Amplitude and then change the color source again.

Since amplitude is an option for gradients, switching to another color source should ignore that option, specially since the checkbox gets disabled when switching.

stuerp commented 8 months ago

I get it. I blocked that scenario in the rendering code but forgot to add the rules to the configuration dialog. Edit: I changed my mind. Switching to a non-gradient color source should not modify a previous selection by the user. How is that different from changing a position of a gradient color? Instead, the rendering code should properly ignore the setting when appropriate.

stuerp commented 8 months ago

The gradient % now stay in place when adding or removing colors, but I'm not being able to edit them by double-clicking anymore. Also, the % doesn't update when changing between visual elements. I have to click again on the selected color to update the box.

A regression because I added double-click to the Presets and an oversight. Both fixed in the next version.

stuerp commented 8 months ago

https://github.com/stuerp/foo_vis_spectrum_analyzer/releases/tag/v0.7.5.0-alpha2