Review Request 110367: Add JoinTheGame menu entry
Albert Astals Cid
aacid at kde.org
Sun Jun 9 11:27:52 BST 2013
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/110367/
-----------------------------------------------------------
(Updated June 9, 2013, 10:27 a.m.)
Status
------
This change has been discarded.
Review request for kdelibs.
Description
-------
Patch by Pau ages ago, Lydia wants to push for it again.
Note that i still find it ugly, but i'm not the kdeui maintainer, so i'll leave it to someone else to discuss.
Not sure if the StandardAction enum addition should be at the end in case someone is storing those enums in a file or something.
Diffs
-----
kdecore/doc/README.kiosk b95002d
kdeui/CMakeLists.txt b439e04
kdeui/actions/kstandardaction.h 3064301
kdeui/actions/kstandardaction.cpp 7de0c6f
kdeui/actions/kstandardaction_p.h b8f8df1
kdeui/dialogs/jointhegame.png PRE-CREATION
kdeui/dialogs/kaboutkdedialog_p.cpp b9728bf
kdeui/dialogs/kjointhegamedialog_p.h PRE-CREATION
kdeui/dialogs/kjointhegamedialog_p.cpp PRE-CREATION
kdeui/shortcuts/kstandardshortcut.h b31e45c
kdeui/shortcuts/kstandardshortcut.cpp 669f750
kdeui/widgets/khelpmenu.h 3389068
kdeui/widgets/khelpmenu.cpp f547c46
kdeui/xmlgui/ui_standards.rc a0f5bed
Diff: http://git.reviewboard.kde.org/r/110367/diff/
Testing
-------
Ran okular, saw the new menu entry, it did stuff.
Ran ksnapshot (that doesn't use xmlgui), saw the new menu entry, it did the same stuff.
Thanks,
Albert Astals Cid
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20130609/6f00d0e0/attachment.htm>
More information about the kde-core-devel
mailing list