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

fix: jumbotron image positioned off-screen #1795

Open
wants to merge 3 commits into
base: src
Choose a base branch
from

Conversation

oliviergoulet5
Copy link
Member

@oliviergoulet5 oliviergoulet5 commented Dec 15, 2023

Fixes bug where bg-secondary jumbotron images are positioned off the screen at certain screen widths.

Copy link

netlify bot commented Dec 15, 2023

Deploy Preview for jakartaee ready!

Name Link
🔨 Latest commit 912a073
🔍 Latest deploy log https://app.netlify.com/sites/jakartaee/deploys/65f98b6b16163e000806dfad
😎 Deploy Preview https://deploy-preview-1795--jakartaee.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 configuration.

@ZacharySabourin
Copy link
Contributor

I'm only seeing a giant orange rectangle. Is that expected?

image

@oliviergoulet5
Copy link
Member Author

Nope. Not sure why I see the img locally. Let me check!

@chrisguindon
Copy link
Member

@oliviergoulet5 Same for me, the image does not load. I suspect it's related to your class name change.

@ZacharySabourin If you have time this week, can you take a look at fixing this MR?

@oliviergoulet5
Copy link
Member Author

Reversed the class name change.

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