Review Request: updated kded4 man page

[David Jarvie]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://svn.reviewboard.kde.org/r/5557/#review8066
-----------------------------------------------------------



/trunk/KDE/kdelibs/doc/kded4/man-kded4.8.docbook
<http://svn.reviewboard.kde.org/r/5557/#comment8304>

    Better:
    "Those tasks are also always performed at ..."



/trunk/KDE/kdelibs/doc/kded4/man-kded4.8.docbook
<http://svn.reviewboard.kde.org/r/5557/#comment8307>

    Subversion Module appears twice (line 145 as well).



/trunk/KDE/kdelibs/doc/kded4/man-kded4.8.docbook
<http://svn.reviewboard.kde.org/r/5557/#comment8305>

    Better: "Provides the sound system ..."



/trunk/KDE/kdelibs/doc/kded4/man-kded4.8.docbook
<http://svn.reviewboard.kde.org/r/5557/#comment8306>

    "network:/" should be enclosed by a tag such as <application> to make it clear it isn't a typo to those not familiar with such syntax.



/trunk/KDE/kdelibs/doc/kded4/man-kded4.8.docbook
<http://svn.reviewboard.kde.org/r/5557/#comment8308>

    Is this for Plasma activities? Clarification would be useful, since "activity" is a generic term.



/trunk/KDE/kdelibs/doc/kded4/man-kded4.8.docbook
<http://svn.reviewboard.kde.org/r/5557/#comment8309>

    nepomuksearch -> "nepomuk search" or "Nepomuk search" ?



/trunk/KDE/kdelibs/doc/kded4/man-kded4.8.docbook
<http://svn.reviewboard.kde.org/r/5557/#comment8310>

    "Provides access to the system's time zones for applications." is more accurate.


- David


On 2010-10-08 21:33:00, Jaime Torres wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://svn.reviewboard.kde.org/r/5557/
> -----------------------------------------------------------
> 
> (Updated 2010-10-08 21:33:00)
> 
> 
> Review request for kdelibs.
> 
> 
> Summary
> -------
> 
> Three questions:
> * Is it wrote in correct English?
> * Is it ok to list the modules in that way and in that page?
> * In the KCM module Service Manager (from where I've copied the names and descriptions) there are 5 descriptions missing (also in the man page):
>   * KDED Subversion module
>   * KDED module for screen-managment
>   * Keyboard Daemon
>   * Display Managment change monitor
>   * K Remote Control Daemon
>   Please, non trivial descriptions?
> 
> 
> This addresses bug 242582.
>     https://bugs.kde.org/show_bug.cgi?id=242582
> 
> 
> Diffs
> -----
> 
>   /trunk/KDE/kdelibs/doc/kded4/man-kded4.8.docbook 1183954 
> 
> Diff: http://svn.reviewboard.kde.org/r/5557/diff
> 
> 
> Testing
> -------
> 
> The man page is more complete, and now can search for out of space
> 
> 
> Thanks,
> 
> Jaime
> 
>

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