Closed proeung closed 1 year ago
aem-code-sync[bot]
commented
1 year ago Hello, I'm Franklin Bot and I will run some test suites that validate the page speed. In case there are problems, just click the checkbox below to rerun the respective action.
aem-code-sync[bot]
commented
1 year ago | Page | Scores | Audits | |
|---|---|---|---|
| /campaigns/curam/missouri |     |
     |
 |
aem-code-sync[bot]
commented
1 year ago | Page | Scores | Audits | |
|---|---|---|---|
| /campaigns/curam/missouri |   |
  |
|
aem-code-sync[bot]
commented
1 year ago | Page | Scores | Audits | |
|---|---|---|---|
| /campaigns/curam/missouri |  |
   |
|
proeung
commented
1 year ago @aavilamer @anabarcelona Can you take a look at the preview (see PR description) and let me know if there are issues that need to be addressed?
Also, I changed the "Helpful Resources" to use the Linkable Cards/Card List pattern since it can handle linking to pages that don't have feature images (see attached).
anabarcelona
commented
1 year ago Hi @proeung, It's looking good! Below are some minor tweeks:
Update the color of the background quarter circle to our Orange 60
Will take some screenshots of the videos and share them. Should I share them in this ticket?
Need link to CCWIS standards
Need hire rez screenshot for the product tour.
Let's remove the 3rd repeated case study in the rotator for now.
Update the color of Helpful resources section to bone
About Curam section looks like the image height needst o be adjusted. If text is too long causing the gap, let's have the copy reduced.
Exciting!!
aem-code-sync[bot]
commented
1 year ago | Page | Scores | Audits | |
|---|---|---|---|
| /campaigns/curam/missouri |  |
|
|
proeung
commented
1 year ago @sharathmrft @anabarcelona Thanks for testing this PR. Please see my response to your findings below.
Testimonial at the bottom is breaking as below
Fixed
2.Inline link should be highlighted.
Fixed
3,Address is missing at the bottom
This is content entry related. We don't need the address in this section.
Update the color of the background quarter circle to our Orange 60
@anabarcelona This is content entry related. These images can be exported and added to Franklin's sharepoint doc when the assets are ready.
Will take some screenshots of the videos and share them. Should I share them in this ticket?
@anabarcelona Please update them in the Figma project and I can export them when it's ready.
Need link to CCWIS standards
Content entry related. Links can be updated in the Sharepoint doc.
Let's remove the 3rd repeated case study in the rotator for now.
Done. Content updated in the Sharepoint doc.
Update the color of Helpful resources section to bone
Done. Content updated in the Sharepoint doc.
About Curam section looks like the image height needst o be adjusted. If text is too long causing the gap, let's have the copy reduced.
The copy of this section can be adjusted in the Sharepoint doc.
aem-code-sync[bot]
commented
1 year ago | Page | Scores | Audits | |
|---|---|---|---|
| /campaigns/curam/missouri | |
 |
|
anabarcelona
commented
1 year ago Hi @proeung, I've updated the design with semi final imagery. We are still waiting on Liz to confirm imagery we've used as well as confirm the final links to the videos.
Please find the latest design updates here: https://www.figma.com/file/lcNLadWizxvzfhBpqds4hU/Curam---State-Microsite?type=design&node-id=664%3A1613&mode=design&t=6h5j4udGRWLhnzCQ-1
proeung
commented
1 year ago @anabarcelona Thanks for the update! I'll export these assets and add them to Franklin, once we get final confirmation from Liz.
aem-code-sync[bot]
commented
1 year ago | Page | Scores | Audits | |
|---|---|---|---|
| /campaigns/curam/missouri | |
|
|
aem-code-sync[bot]
commented
1 year ago | Page | Scores | Audits | |
|---|---|---|---|
| /campaigns/curam/missouri | |
|
|
Issue
Fixes - https://jira.sdlc.merative.com/browse/MERATIVE-880
Description
Changed
Design Specs
Test URLs
main): https://main--merative2--hlxsites.hlx.page/campaigns/curam/missouriTesting Instruction