<div dir="ltr">Hi Miklos,<div><br></div><div>The plan is to:</div><div style> a) rename ctkSpinBox into ctkDoubleSpinBox</div><div style> b) integrate your changes into freshly renamed ctkDoubleSpinBox</div><div style><br>

</div><div style>I just haven't had the time to do the renaming yet.</div><div style><br></div><div style>The reasons behind having ctkSpinBox deriving from QWidget are:</div><div style> a) some QDoubleSpinBox methods are not virtual and can't be reimplemented</div>

<div style> b) it can give greater flexibility if we want to "add" helper widgets in the future</div><div style> c) it prevents the user from calling QDoubleSpinBox methods directly (it is still possible though)</div>

<div style><br></div><div style>Hth,</div><div style>Julien.</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Fri, Jun 14, 2013 at 7:35 AM, 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 guys,<br><br>what should we do with this?<br><br>The current status is that there are two different spin box implementations.<br>

<br>My one is ctkDoubleSpinBox that derives from QDoubleSpinBox and adds the invertedControls properties. It's on a branch in our fork.<br>

<br>The other is ctkSpinBox that derives from QWidget.<br><br>My fix overrides QAbstractSpinBox::stepBy, so it cannot be merged to ctkSpinBox as it is. The ctkSpinBox does not have a stepBy function, although it's part of the public API of QAbstractSpinBox.<br>



<br>I can change ctkSpinBox to use ctkDoubleSpinBox instead of QDoubleSpinBox, and introduce the invertedControls property in ctkSpinBox by delegation.<br><br>But in this way, we would have two spin box classes. (My one could be internal only, but still.)<br>



<br>Is there a specific reason why the ctkSpinBox derives from QWidget and not QDoubleSpinBox? The API of the current implementation is not compatible with the QAbstractSpinBox API what I think, is a problem.<br><br>Cheers,<br>



Miklos<br><br>
</blockquote></div><br></div>