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>

<br>Anyways, I am still looking over for further improvements/suggestions so that I myself can implement those.<br><br>Thanks,<br>Rahul Sharma<br>(NUM_1 on <a href="http://irc.freenode.net">irc.freenode.net</a>)<br><br><div class="gmail_quote">

On Fri, Mar 2, 2012 at 3:31 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">

Hi,<br>
<br>
There is a bug <a href="http://bugs.kde.org/show_bug.cgi?id=152182" target="_blank">http://bugs.kde.org/show_bug.<u></u>cgi?id=152182</a> about the Sliders dialog. A patch is proposed but I am not satisfied with it. Technically the patch works and the student did well. But in my opinion, the problem must be tackled in another way and for that we need KmPlot developers and power users to help.<br>


<br>
I'd like some highlights on the following points about KmPlot Sliders dialog and design decisions, especially point 3):<br>
<br>
1) In the View class we already have 2 methods [void sliderWindowClosed(); and void slidersWindowClosed();] for setting the m_menuSliderAction to false. So instead of adding a new method this should be fixed here. And see why there are 2 such methods.<br>


2) Is there a need for a toggle action as the dialog is either shown or not (doesn't it behave exactly as the Coordinate System dialog?) I would make it a simple action "Show Sliders..." to open the dialog.<br>


3) As I said in the mail when we discussed this bug, the sliders behavior should be clarified. According to <a href="http://userbase.kde.org/KmPlot/Using_Sliders" target="_blank">http://userbase.kde.org/<u></u>KmPlot/Using_Sliders</a> the sliders modify a parameter value. If there is no parameter, the dialog does nothing, as does nothing checking "Slider 1" in the tab. So should this be available at all in those cases?<br>


<br>
Best regards,<br>
<br>
Anne-Marie<br>
<br>
PS: <a href="https://git.reviewboard.kde.org/r/104138/" target="_blank">https://git.reviewboard.kde.<u></u>org/r/104138/</a><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>
</blockquote></div><br>