Ok, I'll look into this. I would inform you soon as I get some hint on the intention of the slider window and also about patch from comment 3.<br><br><div class="gmail_quote">On Fri, Mar 2, 2012 at 8:30 PM, Anne-Marie Mahfouf <span dir="ltr"><<a href="mailto:annemarie.mahfouf@free.fr">annemarie.mahfouf@free.fr</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im">On 03/02/2012 01:10 PM, Rahul Sharma wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Hi Anne-Marie,<br>
First of all, thanks for having a look at my review-request.<br>
I fully understand what you say, and as you had said earlier I had sent a mail to the mailing list regarding this issue.<br>
But I could not get good suggestions on it, and meanwhile I found out why the "close" button didn't behave as expected for toggle, so I sent this patch and Lueck confirmed that the patch worked for him.<br>


</blockquote>
<br></div>
OK so I removed slidersWindowClosed(); to solve 1)<br>
<br>
There is some code to investigate to understand those sliders: in view.cpp, line 3929 we have<br>
void View::updateSliders()<br>
which does things I do not understand.<br>
What's the bool needSliderWindow use?<br>
Could it be that the Close button on the dialog is like "Hide" and that patch from comment 3 in the bug report is right?<br>
If you can explain the above I think we'll know more about the intention of the code.<br>
<br>
Are you up to look into this? I don't think the developer who coded this is still involved in this mailing list so it's up to us to fix this properly.<span class="HOEnZb"><font color="#888888"><br>
<br>
<br>
Anne-Marie</font></span><div class="HOEnZb"><div class="h5"><br>
<br>
<br>
______________________________<u></u>_________________<br>
kde-edu mailing list<br>
<a href="mailto:kde-edu@mail.kde.org" target="_blank">kde-edu@mail.kde.org</a><br>
<a href="https://mail.kde.org/mailman/listinfo/kde-edu" target="_blank">https://mail.kde.org/mailman/<u></u>listinfo/kde-edu</a><br>
</div></div></blockquote></div><br>