Keep route summaries with timelines and simplify tracking layout - #317
Merged
Merged
Conversation
Keep the existing route summary table immediately after its timeline and before orders. Remove the tracking summary heading row and primary metric dividers so wrapped rows align without extra height. Confidence: high Scope-risk: narrow Tested: 112 route tests; build; typecheck; scoped lint; browser ordering in standalone, child and All routes; tracking at 390/681/1440px
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
The route summary table (name, driver, start time and totals) belongs with the timeline. Moving only the timeline left that table below all orders. Place the unchanged summary table immediately after the timeline in standalone, saved child, and All routes views. Tracking visibility remains unchanged.
Remove the Route overview heading and its explanation row. Remove the four primary metrics’ vertical dividers and side indentation; equal column gaps align both rows. Keep the existing evidence disclosure and long-value wrapping.
Validation: 112 route tests, build, typecheck, scoped ESLint, public URL guard and diff check pass. Real-component browser assertions verify map → timeline → summary → orders in all three route views and absence of that summary table on Tracking. At 390/681/1440px, the Tracking summary has no horizontal overflow; its primary metrics have zero left border/padding and aligned column starts. Independent review approved.
Manual K-food deployment follows exact-main CI. No data, route actions, map height or Shopify records change.