KmPlot sliders usage and associated bug

Rahul Sharma rahulveera2009 at gmail.com
Sat Mar 17 04:27:27 UTC 2012


Hi Anne,
Sorry that I could not stay in touch with you for the past 10 days or so
regarding this. Actually I was busy with my exams and other academic loads.
I am resuming my work on this.

Thanks.
Rahul Sharma
(NUM_1 on irc.freenode.net)

On Mon, Mar 5, 2012 at 12:29 AM, Anne-Marie Mahfouf <
annemarie.mahfouf at free.fr> wrote:

> On 03/04/2012 06:56 PM, todd rme wrote:
>
>> On Sun, Mar 4, 2012 at 6:38 PM, Anne-Marie Mahfouf
>> <annemarie.mahfouf at free.fr>  wrote:
>>
>>> On 03/04/2012 09:02 AM, Rahul Sharma wrote:
>>>
>>> Hi,
>>> I looked into the view.cpp updatesliders() function, and I have arrived
>>> at
>>> the conclusion that the variable needSliderWindow does have a use here. I
>>> think that the control if() statement, in which the bool variable
>>> needSliderWindow is set to true, it checks whether the Slider checkbox in
>>> the left panel (function tab) is checked or not, depending on which it
>>> sets
>>> the checkable bool of m_menuSliderAction to true.
>>>
>>> And I think the intention was that when the Slider checkbox is checked,
>>> the
>>> slider window should pop-up and when it is unchecked slider window should
>>> hide.
>>>
>>> So the first thing would be to fix this.
>>> Should closing the Sliders window uncheck the Slider checkbox? What
>>> should
>>> happen when the Sliders window is closed?
>>> Also why is there the possibility to check Slider when there is no Slider
>>> use? Like in a function witn no parameter.
>>>
>>> Anne-Marie
>>>
>>>
>>> And as you were saying there is no need to have that as a toggle. That
>>> can
>>> be done in another way and just setting m_menuSliderAction->**
>>> setchecked(true)
>>> does not fulfill the aim here.
>>>
>>> And one more thing I wanted to point out here is about my patch.
>>> Actually,
>>> when I looked into the code of the ksliderwindow.cpp, I found out that in
>>> order to handle the check of the m_menuSliderAction there is a slot
>>> called
>>> closeevent which emits a windowclosed signal when the window is closed,
>>> which is only called when closeevent is called. But the close button of
>>> the
>>> slider window uses a done() (perhaps inheritted from QDialog as far as I
>>> remember) to close the window which does not emit a closeevent and the
>>> the
>>> m_menuSliderAction becomes faulty, so I just fixed that.
>>>
>>> -Regards
>>>  Rahul Sharma
>>>  (NUM_1 on irc.freenode.net)
>>>
>> Slightly off-topic, but why is this is a window and not a dock panel?
>> Doc panels can be pulled off and used as separate windows, but they
>> can also be integrated into the GUI.
>>
>>  We are trying to find out the use of this Sliders dialog. As I said, it
> is not needed in plots without parameters so a dock for this would take
> space without any purpose. Dialogs do not have the same scope than docks
> and probably only a fraction of users make use of the sliders.
>
> Anne-Marie
>
>
>
>
> ______________________________**_________________
> kde-edu mailing list
> kde-edu at mail.kde.org
> https://mail.kde.org/mailman/**listinfo/kde-edu<https://mail.kde.org/mailman/listinfo/kde-edu>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-edu/attachments/20120317/1a0ccd08/attachment.html>


More information about the kde-edu mailing list