Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Compass is obscured by top bar #869

Closed
squilter opened this issue Jul 9, 2014 · 5 comments
Closed

Compass is obscured by top bar #869

squilter opened this issue Jul 9, 2014 · 5 comments

Comments

@squilter
Copy link
Member

squilter commented Jul 9, 2014

Since the map has been stretched to the top of the screen, the compass in the top left corner has moved up also. It is now under the DP logo and cannot be clicked. It should be farther down.

@m4gr3d
Copy link
Member

m4gr3d commented Jul 9, 2014

Can you add a screenshot of the issue. Thx!

@squilter
Copy link
Member Author

squilter commented Jul 9, 2014

screenshot_2014-07-08-18-16-26

Top left corner

@m4gr3d
Copy link
Member

m4gr3d commented Jul 9, 2014

@squilter I actually never noticed the compass before. I don't think it was visible before the ui update as we didn't account for it in the code. It might have been obscured by the telemetry widget this whole time.
We could fix its padding, or disable it since it wasn't visible before. Fixing it should also caused the 'Google' logo to move as well, as they are both affected by the same padding.

@arthurbenemann @jason4short any preference on fixing it vs disabling it?

@arthurbenemann
Copy link
Member

I'm already doing the padding on the tablet version of the app (which shows the telemetry data on the left side).

It should be keep enabled, because it's an useful feature ( clicking it makes the map return to the north up orientation). @ne0fhyk you probably haven't seen it because it only shows up if you rotate the map.

m4gr3d added a commit to m4gr3d/droidplanner that referenced this issue Jul 9, 2014
arthurbenemann added a commit that referenced this issue Jul 9, 2014
@arthurbenemann
Copy link
Member

Closed by #870

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants