[Konsole-devel] Review Request: Make a 'Configure Konsole' to put all the app wide, non profile-specific, options
Kurt Hindenburg
kurt.hindenburg at gmail.com
Sat Mar 12 00:45:05 UTC 2011
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/100845/
-----------------------------------------------------------
Review request for Konsole.
Summary
-------
There are a few app wide, non profile-specific, options that should be in the main configure dialog. This patch only does 1 as a sample and first attempt at this type of dialog.
One thing that needs cleared up is why settings.cpp and settings/generalsettings.cpp need to be in the konsole__KDEINIT.cpp in CMakeList.txt.
Comments welcomed.
Diffs
-----
src/settings/konsole.kcfg PRE-CREATION
src/settings/settings.kcfgc PRE-CREATION
src/settings/generalsettings.ui PRE-CREATION
src/settings/generalsettings.cpp PRE-CREATION
src/ViewManager.h bf17b4d
src/ViewManager.cpp 56fbec6
src/settings/generalsettings.h PRE-CREATION
src/Profile.h 8aa6dfc
src/Profile.cpp 47e45fc
src/MainWindow.h 2e4276d
src/MainWindow.cpp 046c09a
src/EditProfileDialog.ui 0072f82
src/EditProfileDialog.h e37c77e
src/EditProfileDialog.cpp 073ccc4
desktop/konsoleui.rc 32e222b
src/CMakeLists.txt e17b277
Diff: http://git.reviewboard.kde.org/r/100845/diff
Testing
-------
The one options appears to work OK AFAIK.
Thanks,
Kurt
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/konsole-devel/attachments/20110312/2b774bec/attachment.html>
More information about the konsole-devel
mailing list