> On Aug. 16, 2016, 7:05 a.m., David Faure wrote: > > doc/kcontrol/khtml-general/index.docbook, line 7 > > <https://git.reviewboard.kde.org/r/128684/diff/1/?file=474210#file474210line7> > > > > Yeah I don't understand what this docbook is doing here :-) > > Burkhard Lück wrote: > these docbooks are probably here because plasma-desktop has a copy of > konqueror code containments/desktop/plugins/folder/internallibkonq/?
I doubt that this documentation is related to internallibkonq, which is really just copied here for KonqPopupMenu. Anyhow, konq is changing, don't spend too much time on the documentation right now imho. - David ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/128684/#review98408 ----------------------------------------------------------- On Aug. 15, 2016, 11:40 a.m., Burkhard Lück wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/128684/ > ----------------------------------------------------------- > > (Updated Aug. 15, 2016, 11:40 a.m.) > > > Review request for Documentation, KDE Base Apps, Plasma, and David Faure. > > > Repository: plasma-desktop > > > Description > ------- > > proofread + update > comment webkit > > code in kde-baseapps - docbook in plasma-desktop? > > > Diffs > ----- > > doc/kcontrol/khtml-general/index.docbook 1b9c80e > > Diff: https://git.reviewboard.kde.org/r/128684/diff/ > > > Testing > ------- > > passes checkXML5 > > > Thanks, > > Burkhard Lück > >