fix: map view dynamic cards#6431
Conversation
✅ Deploy Preview for bloom-angelopolis ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
| street: '1120A Madera Avenue', | ||
| city: 'Menlo Park', | ||
| state: 'CA', | ||
| zipCode: '94025', | ||
| latitude: 36.961738, | ||
| longitude: -120.064487, | ||
| latitude: 37.47197516388515, | ||
| longitude: -122.15722059781827, |
There was a problem hiding this comment.
This was adding a pin outside of the bay area region making the map slightly more zoomed out.
✅ Deploy Preview for partners-bloom-dev ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
✅ Deploy Preview for bloom-exygy-dev ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
✅ Deploy Preview for bloom-public-seeds ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
7384d3c to
3d15e9d
Compare
5481b6e to
e067dc3
Compare
|
|
||
| .image-background { | ||
| height: 100%; | ||
| max-height: var(--seeds-s48); |
There was a problem hiding this comment.
Uniform image heights for each card. Cards with more content will now have the image inset
d2f7f2f to
7278ea5
Compare
| } | ||
| } | ||
|
|
||
| h3 { |
There was a problem hiding this comment.
note: The cards take headingClass which would prevent us from having to change this later if we adjust the heading levels structure
There was a problem hiding this comment.
Can you explain what your recommendation here would be? Is it that I create a new class name and put it here while also passing that class into the BloomCard in reference?
There was a problem hiding this comment.
I did what I think you mean, but let me know if it's incorrect
|
@emilyjablonski The change is a recommendation from Hannan. The furthest right version is what was decided. It looks like I need to fine tune the results because there are a couple of cases the images are shorter than the max height when addresses are short and there are no tags |
352584c to
091e6e6
Compare
|
@emilyjablonski this is ready to be re-reviewed. Let me know if you have any questions |
091e6e6 to
fd4ef00
Compare
fd4ef00 to
8ecc639
Compare







This PR addresses #5987
Description
On the listing map search below listing results a set of content cards can now be displayed

This is similar to what Doorway currently has

The three resource cards from the home page will appear here if they are turned on for the jurisdiction. Additionally a new custom content card can be added. This card(s) will display if the text string exist in the general overrides file.
How Can This Be Tested/Reviewed?
This is only visible in the map view so the easiest way to do that is to use the Bridge Bay jurisdiction:
yarn db:setup:staging-regionenableAdditionalResourcesandenableResourcesAuthor Checklist:
yarn generate:clientand/or created a migration when requiredReview Process: