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

[Commit Hook]

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


This review has been submitted with commit 6a5a54de422d8eea01c8ca791809a2839e757515 by Jasem Mutlaq to branch master.

- Commit Hook


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/20131111/1981d7f5/attachment.html>