[Kstars-devel] Review Request 113520: Fixed star density control via mouse-wheel

[Jasem Mutlaq]

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

Ship it!


Ship It!

- Jasem Mutlaq


On Oct. 31, 2013, 3:30 p.m., Vijay Dhameliya wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/113520/
> -----------------------------------------------------------
> 
> (Updated Oct. 31, 2013, 3:30 p.m.)
> 
> 
> Review request for KStars, Samikshan Bairagya, Henry de Valence, and Rafal Kulaga.
> 
> 
> Repository: kstars
> 
> 
> Description
> -------
> 
> -> Problem as pointed by James Bowlin: 
> 
> The problem is that the lower limit of star density available through the mouse-wheel is now very high. It is much higher than what can be seen with the naked eye and is much higher than the default star density when KStars starts for the first time.
> So now if anyone uses <Alt> + <Mouse-Wheel> they will be stuck with a fairly high star density and the only way to get it back to how it started is to go to Preferences --> Configure KStars! For most people, the control is completely broken because it only increases the star density and never decreases it.
> 
> -> Fixed:
> 
> Set upper limit and lower limit of stars density such that it vary from least possible to highest possible value.
> Set magnification factor under <Ctrl> and <Shift> such that stars density increase/decrease linearly rather than exponentially.
> 
> 
> Diffs
> -----
> 
>   kstars/skymapevents.cpp 9cfbb8d 
> 
> Diff: http://git.reviewboard.kde.org/r/113520/diff/
> 
> 
> Testing
> -------
> 
> Tested -> Gives expected results
> 
> 
> Thanks,
> 
> Vijay Dhameliya
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kstars-devel/attachments/20131105/2eecd344/attachment.html>