[kde-doc-english] Review Request 114209: Documentation update for Okteta

Friedrich W. H. Kossebau kossebau at kde.org
Sun Dec 1 15:14:55 UTC 2013


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/114209/#review44934
-----------------------------------------------------------


Thanks for giving the Okteta manual some love again (even if that puts me/us into blame for having ignored it for the least releases, but well).
Changes look good to me, more or less.
There are some more not yet covered items in the menu, which are about view profiles ("View > View Profile" and "Settings > Manage View Profiles"). They are in since 0.10 (SC 4.10) IIRC. Will see to come up with a proposal for a description of those in the next weeks.


doc/index.docbook
<http://git.reviewboard.kde.org/r/114209/#comment32125>

    Is "Ins" standardized over the manuals? Then okay. Otherwise I am not sure if this is a change for the better.
    Because all the english keyboards I saw recently had "Insert", no longer "Ins".
    
    Perhaps there should be KDE-global entities for the different possible keys, for consistent naming :)



doc/index.docbook
<http://git.reviewboard.kde.org/r/114209/#comment32128>

    Yes, is correct. Byte Table was renamed to Value/Char Table.
    
    "both as character and numerical values" possibly should be rephrased to
    "both as character and in the different numerical codings"
    or better.



doc/index.docbook
<http://git.reviewboard.kde.org/r/114209/#comment32129>

    No, is not coupled (currently).



doc/index.docbook
<http://git.reviewboard.kde.org/r/114209/#comment32130>

    Remove unneeded whitespace, please.
    
    Also all the other ones :)



doc/index.docbook
<http://git.reviewboard.kde.org/r/114209/#comment32131>

    Yes, still true also in 0.12 (SC 4.12)



doc/index.docbook
<http://git.reviewboard.kde.org/r/114209/#comment32132>

    Sounds to me as if the selected bytes are copied in all the different formats. 
    
    Perhaps "Copy the selected bytes in one of different formats to select from to the clipboard"? Maybe too complicated.
    
    Also "vailable" -> "available"



doc/index.docbook
<http://git.reviewboard.kde.org/r/114209/#comment32126>

    Unneeded whitespace, please remove


- Friedrich W. H. Kossebau


On Nov. 29, 2013, 5:47 p.m., Burkhard Lück wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/114209/
> -----------------------------------------------------------
> 
> (Updated Nov. 29, 2013, 5:47 p.m.)
> 
> 
> Review request for Documentation, Alexander Richardson and Friedrich W. H. Kossebau.
> 
> 
> Repository: okteta
> 
> 
> Description
> -------
> 
> Update to 4.12
> Replaced the default menu items with link to fundamentals
> 
> @Alex + Friedrich:
> Please review and add comments/fixes/missing info,
> please grep for FIXME to see my questions
> 
> Thanks.
> 
> 
> Diffs
> -----
> 
>   doc/index.docbook 640e30a 
> 
> Diff: http://git.reviewboard.kde.org/r/114209/diff/
> 
> 
> Testing
> -------
> 
> ebn, checkXML,spellcheck
> 
> 
> Thanks,
> 
> Burkhard Lück
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-doc-english/attachments/20131201/53bd1063/attachment.html>


More information about the kde-doc-english mailing list