Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

QA: /contact #186

Closed
arumsey opened this issue Apr 11, 2023 · 4 comments · Fixed by hlxsites/merative2#121
Closed

QA: /contact #186

arumsey opened this issue Apr 11, 2023 · 4 comments · Fixed by hlxsites/merative2#121
Labels
Phase 2 Remaining merative.com site QA Ready for Review
Milestone

Comments

@arumsey
Copy link
Collaborator Author

arumsey commented Apr 14, 2023

@sachinmesh ready for review

@arumsey arumsey self-assigned this Apr 14, 2023
@helms-charity
Copy link

@keith-kaplan and @sachinmesh, this is ready for your review.

@arumsey arumsey removed their assignment Apr 27, 2023
@sachinmesh
Copy link

Hi Team,
Design QA for contact us here

Heading font style should be H1 – Source serif pro 64px light.
Spacing between Title and bodytext should be 24px.
Font styles for Address, Support and Phone should be H4 - Alliance No.1 21px medium.
Submit CTA should have 48px spacing above.

CC: @keith-kaplan @anabarcelona

arumsey referenced this issue in hlxsites/merative2 Apr 28, 2023
- minor fixes to marketo form
@arumsey
Copy link
Collaborator Author

arumsey commented Apr 28, 2023

  • Heading font style should be H1 – Source serif pro 64px light. FIXED
  • Spacing between Title and bodytext should be 24px. FIXED

image

  • Font styles for Address, Support and Phone should be H4 - Alliance No.1 21px medium. FIXED

image

  • Submit CTA should have 48px spacing above. FIXED

image

arumsey referenced this issue in hlxsites/merative2 Apr 28, 2023
* QA: /contact #186

- minor fixes to marketo form

* QA: /events/2023-zelta-user-conference #182

- add `32px` margin to simple content headings
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Phase 2 Remaining merative.com site QA Ready for Review
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants