Skip to content
This repository has been archived by the owner on Jun 3, 2024. It is now read-only.

QMAPS-2129 add startAt / arriveBy options in public transports (draft / poc) #1383

Draft
wants to merge 33 commits into
base: master
Choose a base branch
from

Conversation

xem
Copy link
Contributor

@xem xem commented Jul 27, 2022

  • if public transport is enabled, show 3 forms under the direction form (ugly draft, while waiting for final design!)
    image

  • Pressing "GO" after "start now" updates the URL and computes a route that starts now. (as usual)
    image

  • Choosing a date/time and pressing "GO" after "start at" updates the URL and computes a route that starts after that date/time
    image

  • Choosing a date/time and pressing "GO" after "arrive by" updates the URL and computes a route that arrives before that date/time
    image

NB: I added the date in the routes summaries. It's a suggestion for Design, to remind people which day the route starts

TODO:

  • decide if we want to call the API at any change in the form or after pressing a "ok" submit button (in this PoC it's a button)
  • make a select input to choose between the 2 modes (now/start at/arrive by) (here the 3 are displayed)
  • decide if we want to replace the native datetime input with a JS calendar date input + a hour selector (here, it's the native one)
  • finalize CSS
  • Use zustand to handle / remember the values selected ?

@xem xem marked this pull request as draft July 27, 2022 14:44
pascaloliv and others added 28 commits August 1, 2022 11:26
…ction_for_public_transport

Add distance on walking public transport instructions
* QMAPS-2843 branding

* QMAPS-2843 directions

* QMAPS-2843 directions

* QMAPS-2843 directions

* QMAPS-2843 try to fix github
* QMAPS-2909 fix PT dezoom

* QMAPS-2909 cleanup

* QMAPS-2909 credits due
The topbar was displayed with weird white corners on top and the bottom
corners were also displayed in white instead of transparent.
Make front compatible with intention api field
QMAPS-2850 scroll to selected route
Disable nlu switch and use intention field
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.

4 participants