phlask-map icon indicating copy to clipboard operation
phlask-map copied to clipboard

Update SelectedTap Modal on Desktop to V2 Design

Open tomporvaz opened this issue 2 years ago • 7 comments

Is your feature request related to a problem? Please describe. The SelectedTap modal on desktop should match the new V2 designs. When a user selects resource pin on the map a modal opens on the left side of the screen.

Describe the solution you'd like Make the SelectedTap Modal look like this:

Screenshot from 2023-12-28 13-11-09 https://www.figma.com/file/VGGqwl3Eq3GQIAO9I6LoNW/PHLASK?type=design&node-id=8303-161391&mode=design&t=vVRkRN4MSA1SQjJY-4

Describe alternatives you've considered n/a

Additional context Previously worked on in issue #305, and requires coordination with assignee of issue #361. Some changes may overlap, and coordination is likely required to avoid conflicting changes.

tomporvaz avatar Dec 28 '23 18:12 tomporvaz

@tomporvaz because of how I am implementing this in #361, I am happy to just combine these two tickets into #361 as well. The desktop and mobile versions will use the same content components, and all that changes is how it's displayed (which is already implemented by previous work).

vontell avatar Dec 28 '23 19:12 vontell

Got it! I suspected as much, but I was hoping to make two issues of this. Is there room for two people on this issue? Let's combine it, and if you think there is room for someone to help, we'll add a help wanted flag to #361.

tomporvaz avatar Dec 28 '23 20:12 tomporvaz

Closing this issue to merge with #361. #361 one should implement SelectedTap desktop modal from v2 design in addition to mobile SelectedTab modal.

tomporvaz avatar Dec 28 '23 20:12 tomporvaz

@tomporvaz sounds good! Sure thing, happy to take someone else on who wants to assist/learn!

vontell avatar Dec 29 '23 02:12 vontell

Sorry @tomporvaz! Actually re-opening this so we can have a smaller PR for the current changes. I have a PR open now (#387) and will be working on this next, which should hopefully be trivial.

vontell avatar Jan 17 '24 01:01 vontell

@vontell Implemented much of the logic for the Desktop modal when he built the Mobile modal (issue #361). Take a look around and reach out to Aron if you have questions about the exiting code.

tomporvaz avatar Apr 02 '24 01:04 tomporvaz

I am now taking this on as part of the hackathon

vontell avatar Jun 22 '24 15:06 vontell