[Kstars-devel] Review Request 116965: Added exponential zoom in/out feature in Solar System Viewer

[Rafal Kulaga]

> On April 6, 2014, 10:17 p.m., Rafal Kulaga wrote:
> > Works nicely, but I think it would be better to keep zooming behavior of SSV consistent with skymap - where normally the zoom is operating in coarse mode, but you can lower the zooming step by holding the CTRL key. In your patch, it's the other way around.
> 
> Vijay Dhameliya wrote:
>     Actually zoom operating is so slow i.e. if user wants to zoom enough to view only two inner most orbits it cost many clicks or mouse wheel roles which is bit annoying, at the same time user should be able to lower zooming step like you mentioned, so I would suggest to make use of two different key i.e. Shift and Ctrl to increase and decrease zoom step respectively. What is your opinion about this idea ?
> 
> Rafal Kulaga wrote:
>     Of course, this is possible. But how will you communicate this to the user? CTRL + mouse wheel is common, and I think that it would be a good idea to stick to this pattern.
>     
>     Having said that, I agree that the zoom speed right now is far too slow.
> 
> Vijay Dhameliya wrote:
>     True, using Shift will be of no use as user wont know about it. So should I change the expression which increase the speed such that it decrease the same ?

Yep, just make it other way around and it will be OK.


- Rafal


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


On March 21, 2014, 9:51 p.m., Vijay Dhameliya wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/116965/
> -----------------------------------------------------------
> 
> (Updated March 21, 2014, 9:51 p.m.)
> 
> 
> Review request for KStars and Rafal Kulaga.
> 
> 
> Repository: kstars
> 
> 
> Description
> -------
> 
> Currently zoom in/out factor is set constant which is very small so zoom in/out works very slowly, i.e. user have to keep rolling mouse wheel for long to get zoom enough to observe single planet. 
> 
> This features allows user to do quick zoom in/out by pressing ctrl + wheel/+-keys. 
> 
> 
> Diffs
> -----
> 
>   kstars/tools/pvplotwidget.h ef90ace 
>   kstars/tools/pvplotwidget.cpp b8a8b30 
> 
> Diff: https://git.reviewboard.kde.org/r/116965/diff/
> 
> 
> Testing
> -------
> 
> Testing has done for following use cases:
> 1) Zoom in/out by pressing +/- continuously till it can not zoom in/out furthermore 
> 2) Zoom in/out by rolling rolling wheel up/down continuously till it can not zoom in/out furthermore 
> 3) Zoom in/out by pressing Ctrl + +/- continuously till it can not zoom in/out furthermore 
> 4) Zoom in/out by rolling Ctrl + rolling wheel up/down continuously till it can not zoom in/out furthermore 
> 5) Zoom in/out by pressing +/- step by step till it can not zoom in/out furthermore 
> 6) Zoom in/out by rolling rolling wheel up/down step by step till it can not zoom in/out furthermore 
> 7) Zoom in/out by pressing Ctrl + +/- step by step till it can not zoom in/out furthermore 
> 8) Zoom in/out by rolling Ctrl + rolling wheel up/down step by step till it can not zoom in/out furthermore
> 
> 
> Thanks,
> 
> Vijay Dhameliya
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kstars-devel/attachments/20140407/3bef7b4c/attachment.html>