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

Line route overlay #487

Merged
merged 19 commits into from
Oct 13, 2021
Merged

Line route overlay #487

merged 19 commits into from
Oct 13, 2021

Conversation

Collatty
Copy link
Contributor

@Collatty Collatty commented Oct 13, 2021

Viser ruten til kjøretøyet når du hoverer over det i kartet.

Samme problem som tidligere oppstår her med at man ikke klarer å skille mellom forskjellige avganger på en gitt linje, så den stemmer ikke 100% av tilfellene.

Sånn jeg tenker er dette bra nok til å gå inn i prod, uten at det er lagt opp til innstillinger for det etc. Det kan jobbes videre med i senere PR-er, og vil hindre at vi får en giga-PR mot slutten.

Er tildels avhengig av #484 så den må inn først, da begge benytter seg av state for hover som er implementert på litt forskjellige måter.

image

@Collatty Collatty marked this pull request as ready for review October 13, 2021 09:43
@Collatty Collatty requested a review from a team October 13, 2021 09:43
src/components/Map/index.tsx Outdated Show resolved Hide resolved
@Collatty Collatty merged commit 252406a into master Oct 13, 2021
@magnusrand magnusrand deleted the line-route-overlay branch October 19, 2021 08:03
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 this pull request may close these issues.

2 participants