[Ctk-developers] ctkSliderWidget, ctkDoubleSlider and ctkDoubleSpinBox invertedControls property

[Miklos Espak]

On Fri, May 3, 2013 at 12:56 AM, Julien Finet <julien.finet at kitware.com> wrote:
>
> On Thu, May 2, 2013 at 7:32 PM, Miklos Espak <espakm at gmail.com> wrote:
>>
>> Hi Julien,
>>
>> I am afraid, I cannot do this in the close future.
>>
>> We are still stuck with an older revision of CTK and cannot upgrade
>> because MITK cannot be built with the current CTK master. This is
>> because of a DCMTK CMake issue that has been discussed by JC and Marco
>> recently.
>>
>> So, I do not have a ctkSpinBox yet, and the upgrade would break our build.
>
> I believe you should be able to cherry-pick some commits though.

We tried it with Sascha today, it conflicted. I would resolve it if I
could benefit of the change, but currently, I cannot for the reasons
mentioned before.

I just mentioned that I cannot upgrade the CTK, that means I am *not
able to test* those changes even if I could resolve the conflicts and
build it, which in itself is just double work, with questionable
result and no benefit for me.

>>
>> You said, the ctkSpinBox might lead to some issues. Does this mean
>> that you are thinking of having a separate double and integer spin
>> boxes?
>
> I'm not sure yet, but it's a possibility. I would like to have a good reason
> to do it before doing so. For example, one (false) problem I was thinking of
> was that the ctkSpinBox has signals and slots that take doubles and not
> ints. And sometimes it's much easier when the signals/slots have the right
> type. But we could add those signal/slots to ctkSpinBox, it's not a problem.

In this case, it might be better to wait until the design is finalised.

Cheers,
Miklos