----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/110910/ -----------------------------------------------------------
(Updated June 12, 2013, 8:01 p.m.) Review request for Calligra. Changes ------- Changed context menu initialization to building the context menu right in the constructor. Setters that were previously used to prepare the delayed construction with exec(position) have been turned into constructor arguments. Thanks, Friedrich W. H. Kossebau, for the suggestion (and reminding of the good practice to only write code for present use cases, not for planned things in the future ;) ). Description ------- Refactored context menu creation code out of KoResourceItemChooser and into its own QMenu class, any style feedback is welcome. Also a tiny amount of whitespace cleanup, but only for the files affected by the code changes. Diffs (updated) ----- libs/widgets/KoResourceItemChooser.h 29fb09a libs/widgets/KoResourceItemChooser.cpp 4ae16ad libs/widgets/KoResourceItemChooserContextMenu.h a33a132 libs/widgets/KoResourceItemChooserContextMenu.cpp 6af2fe1 Diff: http://git.reviewboard.kde.org/r/110910/diff/ Testing ------- Tested only for Krita, everything seems to work as it should. Thanks, Sascha Suelzer
_______________________________________________ calligra-devel mailing list [email protected] https://mail.kde.org/mailman/listinfo/calligra-devel
