Gray-able sliders

[Nash]

This simply lets a slider be "gray-able" depending on another CVar's value. The engine already supports this for most other widgets, I am simply exposing the existing behavior to sliders.

Usage is very simple. In your MENUDEF:

Code: Select all

// test_int is the value you want to check for. If this value is 0, the corresponding slider will be grayed out.
// test_float is the slider.
    Option "Test Option", "test_int", "OnOff"
    Slider "Test Slider", "test_float", 0.0, 1.0, 0.1, 1, "test_int"

Pull request
Demo file

[Rachael]

Potentially a very useful option, in my opinion.

[sinisterseed]

I agree, would like to see this too, keeps things consistent for settings that require another to be enabled first.

For on/off toggles this is already there, but sliders are missing.

[Graf Zahl]

Very much appreciated.

[Blue Shadow]

Is this also implemented for the ScaleSlider?

[Nash]

Is this also implemented for the ScaleSlider?

No, I deliberately left that out because ScaleSlider's use case is very specific to that only one thing in GZDoom (the HUD scaling menu). Should I the gray-check to that too? What's the likeliness of modders using that widget?

[Blue Shadow]

I did some coding for someone last year, and that slider type came in handy. Apart from that, I don't know how widely it's used.

My question there was just an inquiry. I wasn't particularly asking for the feature to be added.

[Graf Zahl]

Is this also implemented for the ScaleSlider?

No, I deliberately left that out because ScaleSlider's use case is very specific to that only one thing in GZDoom (the HUD scaling menu). Should I the gray-check to that too? What's the likeliness of modders using that widget?

TBH, if I could implement it without compatibility issues, I'd redo the UI scaling anyway to get rid of the clean scaling, use 1 for displaying at the requested size for the page at hand and use smaller values for downscaling. The current system is awful and only grew out a ZDoom-based design decision from 20+ years ago we did not want to change.

In Raze I completely steered clear of the clean scaling outside the places where the VGA font is used and I think it handles a thousand times better as a result.

Meaning: That slider type is more likely to be deprecated anyway. No need to add support to it.

[Nash]

Yeah, that's what I thought. That's also the other reason I deliberately left ScaleSlider alone; I remember reading a post from you in another thread recently where you said you planned to redo the entire UI scaling stuff.