Skip to content

Conversation

@amy-corson-ibigroup
Copy link
Contributor

Description:
Cleans up the viewed route in the nearby view so that

  • The route overlay is no longer on the map when a user is looking at nearby stops and entities
  • Accessing the nearby view via pattern viewer and then clicking the "back" button in the browser will properly re-set the viewed route and zoom the map correctly (previously it would stay in the nearby view map state)

PR Checklist:

  • Does the code follow accessibility standards (WCAG 2.1 AA Compliant)?
  • Are all languages supported (Internationalization/Localization)?
  • Are appropriate Typescript types implemented?

Copy link
Contributor

@daniel-heppner-ibigroup daniel-heppner-ibigroup left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Let's see if we can find an imperative way to clear the viewed route instead of using a useEffect. Let me know if you want to pair program it.

Copy link
Contributor

@daniel-heppner-ibigroup daniel-heppner-ibigroup left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks! I like this a lot more than the useeffect

Copy link
Contributor

@josh-willis-arcadis josh-willis-arcadis left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@amy-corson-ibigroup amy-corson-ibigroup merged commit bb6afd5 into dev Nov 25, 2025
9 checks passed
@amy-corson-ibigroup amy-corson-ibigroup deleted the clear-viewed-route-in-nearby branch November 25, 2025 18:01
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.

4 participants