Review Request 119120: Fix Next and Last button in Page selector of Report data view to be disabled on last page

Jarosław Staniek staniek at kde.org
Sat Jul 5 19:36:28 BST 2014


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

Ship it!


Good job!

- Jarosław Staniek


On July 4, 2014, 9:52 p.m., Friedrich W. H. Kossebau wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/119120/
> -----------------------------------------------------------
> 
> (Updated July 4, 2014, 9:52 p.m.)
> 
> 
> Review request for Calligra and Jarosław Staniek.
> 
> 
> Repository: calligra
> 
> 
> Description
> -------
> 
> The Next and Last button in the Page selector of the Report data view are always enabled, even if there is only one page or if one is already on the last page.
> 
> This was because the logic for enabling/disabling the next and last button in KexiRecordNavigator::updateButtons(uint recCnt) is also respecting the "isInsertingEnabled" flag, which was not set to "false" for the use with the data view of KexiReportView.
> 
> Setting it explicitely to "false" fixes this for me.
> 
> 
> Diffs
> -----
> 
>   kexi/plugins/reports/kexireportview.cpp 9a21d7a 
>   kexi/widget/utils/kexirecordnavigator.cpp 2f0a8b7 
> 
> Diff: https://git.reviewboard.kde.org/r/119120/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Friedrich W. H. Kossebau
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/calligra-devel/attachments/20140705/2cb22efa/attachment.htm>


More information about the calligra-devel mailing list