This review has been submitted with commit ebc4888fef68b6b9181bd6a07cb40fe46e4b54d1 by Jekyll Wu to branch master.
- Commit
On March 9th, 2012, 5:02 a.m., Jekyll Wu wrote:
|
Review request for Konsole.
By Jekyll Wu.
Updated March 9, 2012, 5:02 a.m.
Description
The existing UI is misleading: the three check boxes imply that you can enable any of them independently, but actually you can only enable one of them at any time. So the three choices should be shown using combo boxes instead of check boxes.
The code is basically stolen from dolphin. Again, I'm always not that confident with UI changes, so please check the code for any stupid or unnecessary code.
|
Testing
Diffs
- desktop/sessionui.rc (0dd1b66)
- src/SessionController.h (46714c0)
- src/SessionController.cpp (f437d9b)
View Diff
Screenshots
|
_______________________________________________
konsole-devel mailing list
konsole-devel@...
https://mail.kde.org/mailman/listinfo/konsole-devel