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

[FEATURE] Adding current location button #131

Open
harsh5902 opened this issue Mar 15, 2023 · 10 comments · May be fixed by #187
Open

[FEATURE] Adding current location button #131

harsh5902 opened this issue Mar 15, 2023 · 10 comments · May be fixed by #187

Comments

@harsh5902
Copy link
Contributor

Description

What kind of feature you are suggesting?

I would like to add a button which will move the map to user's location.

Does it solve any problem?

Usually user tends to locate himself/herself or try to know the distance from his location to other.

Screenshots

No response

Additional information

No response

@nilsnolde
Copy link
Member

I just wanted to close and reference another issue asking for the same thing in #85. But then I realized you were actually the one already asking again for the same thing in #105 😆

We now remember the last map extent (seems like that was also you in #120), I think that's quite convenient too. I could be convinced to ask for a user's location, but for that enough users will have to upvote this issue. So let me leave this open for now.

@harsh5902
Copy link
Contributor Author

Yes, I do understand you merged my pull request recently #120 (last map extent). While working with that issue I realized that you wanted to maintain user's confidentiality and security. But sometimes user wants to locate himself at unknown location or usual menality of the user is to know distace or manuvers to get to other location from his own.

So, for this we can add a prompt for user wether he is comfortable sharing his realtime routing status (or GPS we say) with the app. That would be better solution for it I guess.

@harsh5902
Copy link
Contributor Author

harsh5902 commented Mar 15, 2023

Though I feel that even the last extent of the map is also good but this feature could get so much necessary sometimes
I hope you are getting the real issue user would face without this.
Do let me know your thoughts...

@nilsnolde
Copy link
Member

Yeah I guess you're right, it doesn't need to ask for permission when loading the app, but rather on user request via some
button (ideally in the button toolbar on the right), at which point it'd be the user's decision. So yeah, in the end, I'm with you I think.

@harsh5902
Copy link
Contributor Author

So, suggest what to do, shall I wait for other users feedback or get to start working on it??

@nilsnolde
Copy link
Member

You can if you want. I'm not restricting anyone to work on anything they found themselves, but IMO it's not really necessary to deal with that now, it could be done within the GSoC project (or after).

@ganesh1828
Copy link

Is this issue resolved or still open? can I work on this issue?

@harsh5902
Copy link
Contributor Author

I am on it @ganesh1828 just little busy with clg work.

@spyatmycode
Copy link

I am on it @ganesh1828 just little busy with clg work.

How is it going so far? Have you resolved this issue?

@harsh5902
Copy link
Contributor Author

@nilsnolde I will raise the PR in 5-6 days time, as currently I have university exams going on. That is why it is taking time.

This was referenced Apr 21, 2023
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

Successfully merging a pull request may close this issue.

4 participants