feat: expand the clickable area for project snippet

Plant-for-the-Planet-org / planet-webapp

The Open Source and Commission Free Application that powers the Trillion Tree Campaign. Visit us to meet reforestation Organizations, individuals and companies who are on a journey to plant, monitor and restore a trillion trees.

https://www.trilliontreecampaign.org

Other

37 stars 15 forks source link

feat: expand the clickable area for project snippet #2267

Closed sunilsabatp closed 1 month ago

sunilsabatp commented 1 month ago

[ ] Expand clickable area in project pop-up (on map) so that most of it is included and will lead to project

vercel[bot] commented 1 month ago

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
planet-webapp-multi-tenancy-setup	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Oct 21, 2024 10:11am

2 Skipped Deployments

mohitb35 commented 1 month ago

Not related to this PR, and might have already been addressed:

When I opened /prd/trees-for-peace-an-ayudh-europe-campaign, I saw a client side error:

Screenshot 2024-10-18 at 10 14 50 AM

mohitb35 commented 1 month ago

Not related to this PR, and might have already been addressed:

When I opened /prd/trees-for-peace-an-ayudh-europe-campaign, I saw a client side error:

While this is addressed, there is some doubt about how projects can have null site geometry, and how this is to be handled. @sunilsabatp Could you revert these specific changes, and keep them in a separate PR, which can be worked on/reviewed once the doubts are clarified?

mohitb35 commented 1 month ago

General functionality looks good now. But, compared to the base, the "empty" area between CtA and name of the RO is too big, please check.

Is this resolved?

sunilsabatp commented 1 month ago

General functionality looks good now. But, compared to the base, the "empty" area between CtA and name of the RO is too big, please check.

Is this resolved?

yes