----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/114355/#review45365 -----------------------------------------------------------
Ship it! Looks good to me, some minor issues: What about the global shortcut Alt+Shift+F12 -> Enable/Disable Desktop Effects? Should this be added here? http://docs.kde.org/stable/en/kde-workspace/plasma-desktop/shortcuts.html - replace it with a link to this section? doc/fundamentals/ui.docbook <http://git.reviewboard.kde.org/r/114355/#comment32378> Duplicated ">>": "<link linkend="kbd-docs">>Working ..." -> "<link linkend="kbd-docs">Working ..." - Burkhard L?ck On Dec. 8, 2013, 7:25 p.m., T.C. Hollingsworth wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/114355/ > ----------------------------------------------------------- > > (Updated Dec. 8, 2013, 7:25 p.m.) > > > Review request for Documentation. > > > Repository: kde-runtime > > > Description > ------- > > as discussed on the list earlier > > > Diffs > ----- > > doc/fundamentals/index.docbook e5def05 > doc/fundamentals/ui.docbook e44fc42 > > Diff: http://git.reviewboard.kde.org/r/114355/diff/ > > > Testing > ------- > > passes checkXML and meinprocs successfully > > > Thanks, > > T.C. Hollingsworth > > -------------- next part -------------- An HTML attachment was scrubbed... URL: <http://mail.kde.org/pipermail/kde-doc-english/attachments/20131208/7fbbbab4/attachment.html>
