Skip to content

DBC22-5326: deselect map feature when reversing route on mobile - #1388

Merged
ray-oxd merged 2 commits into
mainfrom
bugfix/DBC22-5326
Jul 6, 2026
Merged

DBC22-5326: deselect map feature when reversing route on mobile#1388
ray-oxd merged 2 commits into
mainfrom
bugfix/DBC22-5326

Conversation

@ray-oxd

@ray-oxd ray-oxd commented Jul 3, 2026

Copy link
Copy Markdown
Collaborator

📝 Submitter

🔗 JIRA Ticket


✅ Quality Assurance & Requirements

  • Requirements Met: I have confirmed that all acceptance criteria from the JIRA ticket are fulfilled.
  • Tested desktop in local or dev envs
  • Tested mobile in local or dev envs
  • Ran unit tests locally
  • SonarCloud: I have verified that the SonarCloud analysis is clean/passing for this branch.

⚙️ Configuration & Environment

  • New Env Variables: Does this PR require new environment variables? (Yes/No)

    If yes, please list them here and ensure they are added to secret manager, the .env.example. and the Vault by an STA.

🧪 How to Test (if required)

  1. On a mobile browser, open the Map and create a route.
  2. Enable the cameras layer and tap a camera icon along the route.
  3. Reverse the route using the swap button.
  4. Verify the selected item panel closes and the map icon is deselected while the new route loads.

🔍 Reviewer Checklist

  • Reviewed code for logic and cleanliness
  • Re-tested desktop/mobile in local or dev envs
  • Verified no new console warnings/errors
  • Confirmed that any new env variables are understood/documented

@ray-oxd
ray-oxd force-pushed the bugfix/DBC22-5326 branch 2 times, most recently from f8f735d to 776057c Compare July 3, 2026 22:29
Clear the clicked feature when route reference params are removed during
a route swap, including swaps initiated from the mobile header search.
@ray-oxd
ray-oxd force-pushed the bugfix/DBC22-5326 branch from 776057c to dbf063c Compare July 6, 2026 22:18
Comment thread src/frontend/src/Components/routing/RouteSearch.js Outdated
@ray-oxd
ray-oxd force-pushed the bugfix/DBC22-5326 branch from dbf063c to a5d7c44 Compare July 6, 2026 22:21
@ray-oxd
ray-oxd merged commit 235515e into main Jul 6, 2026
4 of 5 checks passed
@ray-oxd
ray-oxd deleted the bugfix/DBC22-5326 branch July 6, 2026 22:21
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.

1 participant