Skip to content

Conversation

@mirland
Copy link
Contributor

@mirland mirland commented Dec 9, 2025

issue

Current impl:
image

Fix:

issue.mov

@mirland mirland requested review from Copilot and tomy017 and removed request for Copilot December 9, 2025 14:59
@netlify
Copy link

netlify bot commented Dec 9, 2025

Deploy Preview for blog-xl ready!

Name Link
🔨 Latest commit d7efd0b
🔍 Latest deploy log https://app.netlify.com/projects/blog-xl/deploys/6938720750f6151c78c0c435
😎 Deploy Preview https://deploy-preview-282--blog-xl.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 project configuration.

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR fixes a right menu overlap issue by adjusting responsive layout behavior at the large breakpoint (1200px). The solution hides right-side navigation elements and adjusts content padding to prevent visual overlap on smaller screens.

  • Hides right-side navigation components (title index, social elements, see more buttons) below the lg breakpoint
  • Standardizes padding to 8% on both sides for the lg breakpoint
  • Adjusts image margins to align with the new padding scheme

Reviewed changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated no comments.

Show a summary per file
File Description
src/templates/post.js Adds max-lg:hidden to title index wrapper and adjusts padding/margins to 8% for lg breakpoint
src/styles/global.css Changes bodyPostContainer padding from right-only to horizontal padding at lg breakpoint
src/components/title-blog-index/title-blog-index.jsx Hides both container divs below lg breakpoint (updated from sm)
src/components/social-element/social-element.jsx Hides social element component below lg breakpoint
src/components/see-more-titles/see-more-titles.jsx Hides see more titles button below lg breakpoint

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@mirland
Copy link
Contributor Author

mirland commented Dec 9, 2025

@tomy017, do you know why it's falling? 🤔

@mirland mirland merged commit d293aaa into main Dec 11, 2025
11 checks passed
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