activist-org / activist

An open-source activism platform
https://activist.org
GNU Affero General Public License v3.0
225 stars 185 forks source link

Map option to switch from walking to biking directions #832

Closed andrewtavis closed 3 months ago

andrewtavis commented 5 months ago

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 commented 5 months ago

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

GrandSuccess87 commented 5 months ago

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

andrewtavis commented 5 months ago

Thanks, @GrandSuccess87! Let me know if there's anything I can do to help :) And have fun working on the map! πŸ—ΊοΈπŸ˜Š

andrewtavis commented 5 months ago

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 commented 4 months ago

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 commented 4 months ago

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 commented 4 months ago

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 commented 4 months ago

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 commented 4 months ago

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

GrandSuccess87 commented 4 months ago

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 commented 4 months ago

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

GrandSuccess87 commented 4 months ago

Got it! Tanks for sending those over!

andrewtavis commented 3 months ago

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 :)

GrandSuccess87 commented 3 months ago

@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 commented 3 months ago

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

andrewtavis commented 3 months ago

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