Review Request 128684: Proofread + update khtml-general kcm docbook

David Faure faure at kde.org
Sun Sep 18 18:25:12 BST 2016



> 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
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20160918/64f59d76/attachment.htm>


More information about the kde-core-devel mailing list