Review Request: KAssistantDialog needs to enable/disable back/next/finished buttons after setAppropriate() is called

[Michael Hancock]

> On 2010-05-05 13:52:10, Christoph Feck wrote:
> > Do you have SVN access to commit it? If not, I can apply.

No, I don't have SVN access.


- Michael


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviewboard.kde.org/r/3885/#review5427
-----------------------------------------------------------


On 2010-05-05 12:48:23, Michael Hancock wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://reviewboard.kde.org/r/3885/
> -----------------------------------------------------------
> 
> (Updated 2010-05-05 12:48:23)
> 
> 
> Review request for kdelibs and Olivier Goffart.
> 
> 
> Summary
> -------
> 
> KAssistantDialog currently does not disable/enable the back/next/finished buttons as needed after setAppropriate() is called.  This is needed if the current page sets all of the pages after it to be 'inappropriate', or if it sets all of the previous pages to be 'inappropriate' (though I don't think this case would ever really be used).  I also renamed the method _k_slotCurrentPageChanged() to _k_slotUpdateBackNextFinishedButtons(), since this function is called for more then just the situation where the current page has changed.  This my first time to submit a patch ever (yea, I know this is a pretty small patch), so I would like any feed back I can get. :)
> 
> 
> Diffs
> -----
> 
>   /trunk/KDE/kdelibs/kdeui/dialogs/kassistantdialog.h 1122770 
>   /trunk/KDE/kdelibs/kdeui/dialogs/kassistantdialog.cpp 1122770 
> 
> Diff: http://reviewboard.kde.org/r/3885/diff
> 
> 
> Testing
> -------
> 
> I tested this patch with current kde trunk, and it worked with the kde program I am currently working on (a small program am writing to familiarize my self with KDE development).  With this patch the next/finished buttons now get enabled/disabled after my second page calls setAppropriate() on the pages after it.  I can create a more 'propper' smaller test program if needed. let me know.  
> 
> 
> Thanks,
> 
> Michael
> 
>