<div dir="ltr">Hi Miklos,<div><br></div><div style>We recently added ctkSpinBox to further extend QDoubleBox.</div><div style>In order to have only 1 class to maintain, we decided to have only 1 class in CTK for spinboxes (therefore ctkSpinBox instead of ctkDoubleSpinBox). ctkSpinBox could behave as a QSpinBox if the number of decimals is 0. However I realize it might lead to some issues.</div>

<div style><br></div><div style>Anyway, I don't think it is incompatible with your change. All you have to do is to add the <span style="background-color:rgb(221,255,221);color:rgb(51,51,51);font-family:Consolas,'Liberation Mono',Courier,monospace;font-size:13px;line-height:18px;white-space:nowrap">invertedControls</span> and <span style="background-color:rgb(221,255,221);color:rgb(51,51,51);font-family:Consolas,'Liberation Mono',Courier,monospace;font-size:13px;line-height:18px;white-space:nowrap">invertedAppearance</span> properties to ctkSpinBox. Then you should be able to make a pull request.</div>

<div style><br></div><div style>Thanks,</div><div style>Julien.</div><div style><br></div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Thu, May 2, 2013 at 5:34 PM, Miklos Espak <span dir="ltr"><<a href="mailto:espakm@gmail.com" target="_blank">espakm@gmail.com</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Julien,<br>
<br>
I added an invertedAppearance an invertedControls property to the<br>
ctkDoubleSlider and ctkSliderWidget classes.<br>
<br>
The QSlider has already these properties, so most of the code is just<br>
delegation. The QDoubleSpinBox, that was used internally by<br>
ctkSliderWidget, however, does not have an invertedControls property.<br>
<br>
So, I derived a ctkDoubleSpinBox from QDoubleSpinBox to implement this<br>
feature. Then, I replaced the spin box of the slider widget from the<br>
Qt one to this one.<br>
<br>
All works fine, but I cannot send a pull request, because somebody<br>
else has also replaced the spin box of the sliderwidget with a custom<br>
class (ctkSpinBox), recently.<br>
<br>
I pushed my tweak to our fork here:<br>
<br>
<a href="https://github.com/NifTK/CTK/commit/f1a2e0e62e1c6285810c84685ca945675e9e16fd" target="_blank">https://github.com/NifTK/CTK/commit/f1a2e0e62e1c6285810c84685ca945675e9e16fd</a><br>
<br>
Branch name: slider-inverted-properties<br>
<br>
Qt has a separate QSpinBox and QDoubleSpinBox class, but as I see, the<br>
ctkSpinBox is already for doubles. I do not know what would be the<br>
best way to resolve this conflict.<br>
<br>
Could you please review the change and merge it if you agree?<br>
<br>
Thank you,<br>
Miklos<br>
_______________________________________________<br>
Ctk-developers mailing list<br>
<a href="mailto:Ctk-developers@commontk.org">Ctk-developers@commontk.org</a><br>
<a href="http://public.kitware.com/cgi-bin/mailman/listinfo/ctk-developers" target="_blank">http://public.kitware.com/cgi-bin/mailman/listinfo/ctk-developers</a><br>
</blockquote></div><br></div>