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

Adds responsive component learn part 2 #87

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

rodrickcalvin
Copy link
Contributor

@rodrickcalvin rodrickcalvin commented Apr 11, 2022

Updates code to match Figma design of issue#42 and makes section responsive

@netlify
Copy link

netlify bot commented Apr 11, 2022

Deploy Preview for orcahome ready!

Name Link
🔨 Latest commit 7747e2a
🔍 Latest deploy log https://app.netlify.com/sites/orcahome/deploys/6254b4defb329b00091f338f
😎 Deploy Preview https://deploy-preview-87--orcahome.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.

@UXBrendan
Copy link

UI check for responsiveness passes.

@UXBrendan UXBrendan requested review from paulcretu and removed request for evanjscallan September 29, 2023 01:45
@UXBrendan
Copy link

@rodrickcalvin

We are going to deploy all passed PRs this Tuesday. Would it be possible for you to pass this PR so it's ready for our deployment session on Tuesday, Oct. 17?

@rodrickcalvin
Copy link
Contributor Author

@UXBrendan I don't have the rights to pass the PR. The repository was setup in a way that someone has to review first as a flag then on review, the checks can be passed.

<div className={learnStyles.imageWrapper}>
<section aria-labelledby="salish" className={learnStyles.salishSection}>
<h1 id="salish" className={learnStyles.salishHeading}>
{' '}

Choose a reason for hiding this comment

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

Is this preceding space necessary in the H1?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants