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

Map option to switch from walking to biking directions #832

Closed
2 tasks done
andrewtavis opened this issue Apr 22, 2024 · 16 comments
Closed
2 tasks done

Map option to switch from walking to biking directions #832

andrewtavis opened this issue Apr 22, 2024 · 16 comments
Assignees
Labels
feature New feature or request help wanted Extra attention is needed

Comments

@andrewtavis
Copy link
Member

andrewtavis commented Apr 22, 2024

Terms

Description

Now that work has been finished on #685, we're now ready to provide an option for the user to choose their routing options. The code to switch the profile is currently commented out in the MediaMap component, with the base designs for this being found on Figma. For this first issue, it'd be great if we gave the user the option to switch between walking and biking direction services, with another issue then following that would then add the full functionality of switching between walking, biking and driving. The default would be walking, and if the user clicks the walking icon, then the api and directions profile would be switched to the biking directions.

Contribution

Happy to support with this or get to it myself eventually! 😊

@andrewtavis andrewtavis added feature New feature or request help wanted Extra attention is needed labels Apr 22, 2024
@andrewtavis
Copy link
Member Author

CC @GrandSuccess87, would you have interest in working on this?

@GrandSuccess87
Copy link
Contributor

Hey @andrewtavis, I'm happy to work on this!

@andrewtavis
Copy link
Member Author

Thanks, @GrandSuccess87! Let me know if there's anything I can do to help :) And have fun working on the map! 🗺️😊

@andrewtavis
Copy link
Member Author

andrewtavis commented Apr 23, 2024

Further question that I have on this for maybe another issue or two, @GrandSuccess87:

What about adding in a tooltip or something to the second waypoint that would show the time estimate for how long it'd take to get there? I'm realizing that the route is often times going to be the same for biking and walking, but having a time estimate would be a major distinction for it 😊

Beyond that, let's keep in mind that we'll likely want to clear there directions when the profile is changed to make sure that the user redraws the path and gets accurate directions with the new one. Would likely be ideal if we were able to reroute, but that's a separate issue as well.

And a quick check on your opinion on something: should we add the route directions to the path, so the little arrows that are on the path? Might be obvious, but I'm seeing that Organic Maps does this :)

@andrewtavis
Copy link
Member Author

Hey @GrandSuccess87 👋 Checking in with you on the issue :) Let me know if you want to do a call at some point to discuss the status! Hope you're well! 😊

@GrandSuccess87
Copy link
Contributor

Hii @andrewtavis! Apologies! I've had some unexpected work demands popup over the last several months and as such I'm just now getting started on this + caught up on all the latest updates from the past dev syncs 🥲

I do like the idea of adding a tool tip to the second wave point to show the time estimate as a means of distinguishing the walking / biking routes.

I also like the idea of showing the arrows in the route directions since it'll promote clarity to the user 💯

Do we want the tool tip + addition of route directions to the path in separate tickets or part of this one? If separate tickets, I can create those too :))

@andrewtavis
Copy link
Member Author

andrewtavis commented May 18, 2024

Great to hear from you, @GrandSuccess87! We're still in Code Night 🌙 if you'd want to stop in and say hi :) You can probably guess who all's still in the call 😊

@andrewtavis
Copy link
Member Author

Do we want the tool tip + addition of route directions to the path in separate tickets or part of this one? If separate tickets, I can create those too :))

I think that making separate tickets would be great for that :) Feel free to create them, and thank you!

@GrandSuccess87
Copy link
Contributor

Wonderful! I'm at the cafe, I'll pop in and say hello!

@GrandSuccess87
Copy link
Contributor

GrandSuccess87 commented May 24, 2024

Hey @andrewtavis! Would you happen to have a solo bike icon image I can use by any chance? I checked the Figma, and I see it's actually grouped with the others that we plan to add later.

Screenshot 2024-05-23 at 10 45 23 PM

@andrewtavis
Copy link
Member Author

Hey @GrandSuccess87 👋 The bike icon there can be found at icones.js.org/collection/bi?s=bic&icon=bi:bicycle :) In case you'd like the walking one too, that's icones.js.org/collection/bi?s=walk&icon=bi:person-walking, but then I flipped it over the y-axis so the person's walking to the left 😉 Attaching an SVG for that as well :)

bi_person_walking_left

@andrewtavis andrewtavis moved this from Todo to In Progress in activist Board May 24, 2024
@GrandSuccess87
Copy link
Contributor

Got it! Tanks for sending those over!

andrewtavis added a commit to GrandSuccess87/activist-TA that referenced this issue Jun 29, 2024
andrewtavis added a commit to GrandSuccess87/activist-TA that referenced this issue Jun 30, 2024
andrewtavis added a commit that referenced this issue Jun 30, 2024
…alking-To-Biking

#832/feature map option walking to biking
@andrewtavis
Copy link
Member Author

Closed by #901 🥳 Thanks so much for the dedication here, @GrandSuccess87! Really is such a great feature, and is only gonna get better 😊 Let me know if you have ideas for a next issue. Also happy to suggest if need be :)

@github-project-automation github-project-automation bot moved this from In Progress to Done in activist Board Jun 30, 2024
@GrandSuccess87
Copy link
Contributor

@andrewtavis So happy to have been a part of working on this feature 😀 I'm happy to work on #894 next or if there's another ticket you have in mind, happy to check that out!

@andrewtavis
Copy link
Member Author

That'd be a good one to check out, @GrandSuccess87 :) Want to write in there so I can assign?

@andrewtavis
Copy link
Member Author

Hehe nvm, you made the issue :) Assigned, and let me know if I can help!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature New feature or request help wanted Extra attention is needed
Projects
Archived in project
Development

No branches or pull requests

2 participants