Review Request 125878: fix lost QuickViewSharedEngine initialSize on initializing

Aleix Pol Gonzalez aleixpol at kde.org
Fri Oct 30 10:31:46 UTC 2015


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


+1

- Aleix Pol Gonzalez


On Oct. 30, 2015, 6:07 a.m., Takahiro Hashimoto wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/125878/
> -----------------------------------------------------------
> 
> (Updated Oct. 30, 2015, 6:07 a.m.)
> 
> 
> Review request for Plasma, David Edmundson and David Faure.
> 
> 
> Repository: kdeclarative
> 
> 
> Description
> -------
> 
> After my commit 1c3f42afe17cc090372eaf45647d71495de12c19, view's initialiSize value is lost (remaining 0,0 in spite of item's size). It should be set by item's size for some reusing purpose (with initialSize() getter) 
> 
> Sorry again my bad commit :(
> 
> 
> Diffs
> -----
> 
>   src/quickaddons/quickviewsharedengine.cpp 323cf92 
> 
> Diff: https://git.reviewboard.kde.org/r/125878/diff/
> 
> 
> Testing
> -------
> 
> build and run test
> 
> kenya888 at xps13 ~/devel/kf5/build/frameworks/kdeclarative $ ninja test
> [1/1] Running tests...
> Test project /home/kenya888/devel/kf5/build/frameworks/kdeclarative
>     Start 1: fullmodelaccesstest
> 1/2 Test #1: fullmodelaccesstest ..............   Passed    0.05 sec
>     Start 2: quickviewsharedengine
> 2/2 Test #2: quickviewsharedengine ............   Passed    0.39 sec
> 
> 100% tests passed, 0 tests failed out of 2
> 
> Total Test time (real) =   0.44 sec
> 
> ------
> 
> confirmed ksplashqml works well which use QuickViewSharedEngine.
> 
> 
> Thanks,
> 
> Takahiro Hashimoto
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20151030/b1371579/attachment.html>


More information about the Plasma-devel mailing list