Intersection Map UI Update #174
Conversation
…buttons/formatting
| <MapLegend /> | ||
| <VisualSettings /> |
There was a problem hiding this comment.
These can override each other - can we have them toggle or add them to a dynamic grid? This includes the visual settings, information, and legend
There was a problem hiding this comment.
Added a toggle so that these no longer display over one another
| setOpen(!open) | ||
| }} | ||
| > | ||
| <VpnKeyOutlined /> |
There was a problem hiding this comment.
I don't love this as a legend icon
There was a problem hiding this comment.
Maybe something more like this (from wyoroadinfo):
There was a problem hiding this comment.
Updated to use an icon similar to the one you suggested
| right: 0, | ||
| width: open ? 600 : 50, | ||
| fontSize: '16px', | ||
| overflow: 'auto', |
There was a problem hiding this comment.
The scrollbar should be for the contents, and leave the title at the top:
There was a problem hiding this comment.
This applies to the other popup components as well
There was a problem hiding this comment.
Updated the popup components to only scroll content
PR Details
This PR updates the intersection map UI to match the following figma prototype: https://www.figma.com/proto/QG64ugNXpXMUk2vQ1wgPdZ/CV-Manager-Prototype-January-2025?page-id=2143%3A6061&node-id=2146-7749&viewport=-595%2C-659%2C0.22&t=A1CuORABoO0Qexzk-1&scaling=scale-down&content-scaling=fixed&starting-point-node-id=2146%3A6646
Description
This PR includes the following changes:
How Has This Been Tested?
This was tested locally with docker compose and unit tests were verified to pass using npm. Please note that these changes were tested with the theme updates present in #172.
Types of changes
Checklist: