Zone Details: Header part 1 #23
Conversation
|
Github breaks because I added so many flags, i will do that separately |
|
A reminder for future: I think we should test UX of this carefully when we're further, as I'm a bit concerned of things jumping a lot between different zones and making comparison/scanning harder due to the "badges" being placed like this. Maybe it's not an actual problem, so let's not do something about it now - just want to make sure we remember to play around with this later :) |
| // TODO: add correct icon | ||
| return ( | ||
| <div className="flex flex-row pl-2"> | ||
| <Link className="mr-4 self-center text-3xl text-gray-400" to={goBackPath}> |
There was a problem hiding this comment.
Is there any use case where we use the goBackPath for anything but /map? Otherwise I'd remove the prop and just hardcode it
There was a problem hiding this comment.
Good idea, we can always bring it back
Rasmus and I discussed this as well, the only thing jumping really would be the labels which we can quickly switch with the data. This design allows for any number of labels which may prove useful in the future. |
Step in ELE-1341
Based on the updated visuals provided by Rasmus, which also changes the functionality slightly