search

rt-bishop / Look4Sat

Open-source satellite tracker and pass predictor for Android, inspired by Gpredict
https://play.google.com/store/apps/details?id=com.rtbishop.look4sat
GNU General Public License v3.0
613 stars 58 forks source link

Don't show aiming icon when "Use sensors to rotate PolarView" is disabled #57

Closed shuyuan-liu closed 3 years ago

shuyuan-liu commented 3 years ago

First of all, thanks for adding the aiming feature - very useful for hand tracking satellites. However I think the icon should only be shown when "Use sensors to rotate PolarView" is enabled, because otherwise it is fixed at 0° due north and serves no purpose.

In the screenshot below I have disabled rotating PolarView, causing the aim icon to be fixed at the top of the PolarView:

image

rt-bishop commented 3 years ago

@shuyuan-liu makes perfect sense, thanks for reporting this. I've done like you requested as well as fixed the bugs with the recent 2.5.2 update. I'm glad people find the aiming feature useful. It was a lot easier to implement than I expected, wish I did it earlier.

shuyuan-liu commented 3 years ago

Thanks @rt-bishop, yes the aiming is quite handy. My phone's accelerometer is off by quite a few degrees though, will need to look for a way to calibrate it.

Reported problem solved, closing the issue