Page 1 of 1

Numbering of Playback sliders

Posted: Mon Aug 03, 2020 1:05 am
by jpearson0525
The number of the Cue Stack playback sliders has two # 15 sliders.

Version 4.12.2, Raspbian Buster based on Raspberry Pi 3 B+.

JIm.

Re: Numbering of Playback sliders

Posted: Wed Aug 05, 2020 2:01 am
by kenact
t does in Windows too.

Re: Numbering of Playback sliders

Posted: Wed Aug 05, 2020 7:41 am
by edogawa
I think I found where in the code that happens, it's just an issue with displaying, internal numbering is ok.

I just recompiled an it displays correctly now. Before, at resizing the window, the doubled number always appeared when a new playback slider got added, that led me to looking for resize code.

I don't have time today to make a proper Github pull request, but as it's such a trivial change, maybe @Massimo can you just pick it up from here please?

ui/src/simpledesk.cpp, line 1681 should add 1 to the slider count.

Code: Select all

                slider->setLabel(QString::number(m_playbackSliders.count()));
should be changed to

Code: Select all

                slider->setLabel(QString::number(m_playbackSliders.count()+1));

Re: Numbering of Playback sliders

Posted: Wed Aug 05, 2020 9:49 am
by mcallegari
Sure, I'll have a look at this maybe later today.
Thanks for reporting

[EDIT] just pushed upstream