Review Request 125565: KMail: Fix scrolling up/down on the message viewer

Jeremy Whiting jpwhiting at kde.org
Fri Oct 9 02:06:03 UTC 2015


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/125565/#review86523
-----------------------------------------------------------


Explanation makes sense to me. I think a pim person needs to give the ship it though.

- Jeremy Whiting


On Oct. 8, 2015, 6:39 p.m., Albert Astals Cid wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/125565/
> -----------------------------------------------------------
> 
> (Updated Oct. 8, 2015, 6:39 p.m.)
> 
> 
> Review request for KDEPIM and Release Team.
> 
> 
> Repository: kdepim
> 
> 
> Description
> -------
> 
> The new style connection for signals does not support default arguments so 
> 
> connect(mScrollUpAction, &QAction::triggered,
>             q, &Viewer::slotScrollUp);
>             
> Connects the version of triggered(bool) with slotScrollUp meaning that slotScrollUp always gets a 0 since the action is never checked.
> 
> This breaks the API but the widget is only used internally so i think it's something we can live on.
> 
> Release-team can we sneak this onto 15.0.8.2 since it's relatively critical?
> 
> kdepim guys is there a bug number for this?
> 
> 
> Diffs
> -----
> 
>   messageviewer/viewer/viewer.h 150c24b 
>   messageviewer/viewer/viewer.cpp ea4fa75 
> 
> Diff: https://git.reviewboard.kde.org/r/125565/diff/
> 
> 
> Testing
> -------
> 
> WOOOOOOOOOOOOORKS
> 
> 
> Thanks,
> 
> Albert Astals Cid
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/release-team/attachments/20151009/c8610dd4/attachment.html>


More information about the release-team mailing list