Skip to content

#14 - Add "For Technical Assistance" Content To "Our Services" Page#45

Merged
dhyeypatell merged 3 commits intodevelopfrom
feat/14-add-page-bottom-cta-content-to-our-services-page
Nov 21, 2022
Merged

#14 - Add "For Technical Assistance" Content To "Our Services" Page#45
dhyeypatell merged 3 commits intodevelopfrom
feat/14-add-page-bottom-cta-content-to-our-services-page

Conversation

@dhyeypatell
Copy link
Copy Markdown
Contributor

Description

  • Used CallToAction component to add "For Technical Assistance" Content to "Our Services" Page

Checklist

  • PR self-review by the PR creator
  • Tested by the PR creator
  • Tested by the PR reviewer

Special Test Notes

To test the responsiveness of CallToAction, note the following breakpoints in terms of screen width:

  • 1199 px
  • 991 px
  • 767 px
  • 388 px
  • 315 px

Reminders

  • Make sure you're merging the PR into the correct branch.

@dhyeypatell dhyeypatell added the enhancement New feature or request label Nov 21, 2022
@dhyeypatell dhyeypatell self-assigned this Nov 21, 2022
@netlify
Copy link
Copy Markdown

netlify bot commented Nov 21, 2022

Deploy Preview for fanciful-manatee-3a0823 ready!

Name Link
🔨 Latest commit 0e1eb86
🔍 Latest deploy log https://app.netlify.com/sites/fanciful-manatee-3a0823/deploys/637bea180368da0009ed8450
😎 Deploy Preview https://deploy-preview-45--fanciful-manatee-3a0823.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@mistryrn
Copy link
Copy Markdown
Collaborator

Looks good, might just need a minor tweak to the styling of the button. For reference, here's the CTA button on Prod:
Screen Shot 2022-11-21 at 3 02 19 PM

And here's what I'm seeing with the branch currently checked out:
Screen Shot 2022-11-21 at 3 02 09 PM

@dhyeypatell
Copy link
Copy Markdown
Contributor Author

Looks good, might just need a minor tweak to the styling of the button. For reference, here's the CTA button on Prod: Screen Shot 2022-11-21 at 3 02 19 PM

And here's what I'm seeing with the branch currently checked out: Screen Shot 2022-11-21 at 3 02 09 PM

I have improved the paddings as requested.

@dhyeypatell dhyeypatell reopened this Nov 21, 2022
Copy link
Copy Markdown
Collaborator

@mistryrn mistryrn left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good! 👍

@dhyeypatell dhyeypatell merged commit d765855 into develop Nov 21, 2022
@dhyeypatell dhyeypatell deleted the feat/14-add-page-bottom-cta-content-to-our-services-page branch November 21, 2022 21:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants