Review Request 110452: Change API doc for KTextBrowser

Kevin Ottens ervin at kde.org
Thu May 16 06:25:14 UTC 2013


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


Don't worry about mailClick, looking with lxr.kde.org the only use is in kopete.


staging/kde4support/src/kdeui/ktextbrowser.h
<http://git.reviewboard.kde.org/r/110452/#comment24271>

    Spurious space



staging/kde4support/src/kdeui/ktextbrowser.h
<http://git.reviewboard.kde.org/r/110452/#comment24272>

    I think it's ok to keep that block.


- Kevin Ottens


On May 15, 2013, 4:34 p.m., Anne-Marie Mahfouf wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/110452/
> -----------------------------------------------------------
> 
> (Updated May 15, 2013, 4:34 p.m.)
> 
> 
> Review request for KDE Frameworks and David Faure.
> 
> 
> Description
> -------
> 
> Change API doc in KTextBrowser: mark as deprecated, inform how to port.
> 
> I could not find out what QTextBrowser signal to use instead of KTextBrowser::mailClick though.
> 
> 
> Diffs
> -----
> 
>   staging/kde4support/src/kdeui/ktextbrowser.h 5e5daf9 
> 
> Diff: http://git.reviewboard.kde.org/r/110452/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Anne-Marie Mahfouf
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20130516/a5a368cf/attachment.html>


More information about the Kde-frameworks-devel mailing list