Review Request: Use KAssistantDialog instead of QWizard for the new class assistant.
Ghislain Mary
gmary at lunacymaze.org
Thu Apr 12 08:11:36 UTC 2012
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/104547/
-----------------------------------------------------------
(Updated April 12, 2012, 8:11 a.m.)
Review request for KDevelop.
Changes
-------
Added the requested screenshots.
Description
-------
Use KAssistantDialog instead of QWizard for the new class assistant and use the "assistant" word instead of "wizard".
I tried to make as less changes as possible. I put the subtitle of the pages in the title of a groupbox and added an icon for each page.
This addresses bug 181691.
http://bugs.kde.org/show_bug.cgi?id=181691
Diffs
-----
language/codegen/createclass.h 47b39f2378db24815f7a79b48bbd6c2a8211cf99
language/codegen/createclass.cpp e52107f9d198f144d9d27a33a252b574cc6ed979
language/codegen/overridespage.h 7bf3832fdc81543fd90ce7aec6b12bf32da97839
language/codegen/overridespage.cpp 1a64f3de4ba4255c83fae23ef52a80b1071ca9ab
language/codegen/ui/licensechooser.ui 21bea9476d875c9b56d883e569c09c08da1e89b3
language/codegen/ui/newclass.ui 760eef6eed5c24ddbf0a2249450d1bd083c8c490
language/codegen/ui/outputlocation.ui 09110cc01a74c5f938f3df8b3e21e326ffc11280
language/codegen/ui/overridevirtuals.ui dbf9abe7053dfd9403f6b788bf33192aa553074b
Diff: http://git.reviewboard.kde.org/r/104547/diff/
Testing
-------
Created several classes, overriding some classes or not.
Also tried to go back in the assistant to make changes on one of the previous pages.
Screenshots (updated)
-----------
Comparison of QWizard and KAssistantDialog for the new class assistant
http://git.reviewboard.kde.org/r/104547/s/523/
Thanks,
Ghislain Mary
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdevelop-devel/attachments/20120412/933e5886/attachment.html>
More information about the KDevelop-devel
mailing list