[Konsole-devel] Review Request 110318: Improvements in ColorSchemeEditor dialog

[renan fargetton]

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

Review request for Konsole.


Description
-------

Improvements in ColorSchemeEditor dialog :
- colorTable has one more row for intense colors. This way each colors and the corresponding intense color is side by side (the data model colorScheme is unchanged, only the ui has the additional column)

- colorTable has fixed height such that the table fit exactly inside.

- an Apply button is added, so that it is possible to save modified colorscheme directly from colorSchemeEditor dialog. This way it is easier to preview the result of the theme change, no need to close/open the dialog to get the preview. 
       * Most of the changes are in EditProfileDialog.cpp where the Kdialog is set up (EditProfileDialog::showColorSchemeEditor() ).
       * Saving is done in a separated slot EditProfileDialog::saveColorScheme(const ColorScheme& scheme)


Diffs
-----


Diff: http://git.reviewboard.kde.org/r/110318/diff/


Testing
-------

Run it on my machine / create and modify colorSchemes with the dialog


Thanks,

renan fargetton

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/konsole-devel/attachments/20130505/680b2d8d/attachment.html>