Skip to content

Conversation

@akardet
Copy link
Collaborator

@akardet akardet commented Apr 6, 2023

Adding legal links to footer

@akardet akardet self-assigned this Apr 6, 2023
@netlify
Copy link

netlify bot commented Apr 6, 2023

Deploy Preview for jamstack-site ready!

Name Link
🔨 Latest commit fbca637
🔍 Latest deploy log https://app.netlify.com/sites/jamstack-site/deploys/642f0a897b32390008b5be7a
😎 Deploy Preview https://deploy-preview-879--jamstack-site.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.

@stephan1echung
Copy link

screenshot

@akardet Let's remove the Trust Center link so there's not a lone link at the bottom

Browser metadata
Path:      /
Browser:   Chrome 111.0.0.0 on Mac OS 10.15.7
Viewport:  1920 x 824 @2x
Language:  en-US
Cookies:   Enabled

Open in BrowserStack

Open Deploy Preview · Mark as Resolved

@akardet
Copy link
Collaborator Author

akardet commented Apr 6, 2023

@akardet Let's remove the Trust Center link so there's not a lone link at the bottom

@stephan1echung - I changed it to 3 columns so it lays out better

@stephan1echung
Copy link

stephan1echung commented Apr 6, 2023

screenshot

@akardet ooh good idea! Are the columns centered though? The embedded video is throwing me off 🧐

Browser metadata
Path:      /
Browser:   Chrome 111.0.0.0 on Mac OS 10.15.7
Viewport:  1920 x 824 @2x
Language:  en-US
Cookies:   Enabled

Open in BrowserStack

Open Deploy Preview · Mark as Resolved

@akardet
Copy link
Collaborator Author

akardet commented Apr 6, 2023

@stephan1echung - I'd say it's fine to leave as is, as the words break better for all viewports so it behaves more responsively.

Copy link

@stephan1echung stephan1echung left a comment

Choose a reason for hiding this comment

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

Got it, sounds good then let's merge! :shipit:

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.

3 participants