Skip to content
This repository has been archived by the owner on Mar 4, 2019. It is now read-only.

Levelling tool v2. #75

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Levelling tool v2. #75

wants to merge 4 commits into from

Conversation

andree182
Copy link

Improved size, color and positioning of the levelling circle.

The circle changes size as it moves from dead-center and opacity, as it moves
away from either axis.

Also, now it always stays withing the compass circle.
@gdt
Copy link

gdt commented May 31, 2016

It looks like this PR has several features which aren't related. I think it makes sense to show satellite locations for any being tracked, even without fix, but I don't see how that's related to leveling.

@gdt
Copy link

gdt commented May 31, 2016

Please consider making the leveling circle a preference (and default to off). I use satstat to see information about GPS fix quality, to force GPS on to preload ephemeris, and to visualize location providers. For all of this I hold my phone so I can read it, not trying to make it level. Perhaps if the level only comes on when the device is close, that would avoid the issue.

@andree182
Copy link
Author

Counterargument - it's a feature that doesn't clutter the satellite view too much, and may be helpful on some occasions. Satstat can even show maps, so visualising the orientation is IMO not really that big addition...

But sure, if it's even default off, I don't really care in the end. I just don't want to maintain a parallel fork of satstat, unless there's a big uprising against this :-)

@mvglasow
Copy link
Owner

mvglasow commented Jul 9, 2016

@andree182 are you planning to switch to using the accelerometer, as announced in #76? Should I hold the pull request back until then?

@mvglasow mvglasow added this to the 3.1 milestone Jul 17, 2016
@mvglasow mvglasow modified the milestones: 3.2, 3.1 Jul 27, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants