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

[Bug] Pagination padding and full-width margins creating horizontal scroll on mobile #357

Closed
joe9663 opened this issue Apr 6, 2021 · 1 comment · Fixed by #517
Closed
Labels
bug General bug issues Crio
Milestone

Comments

@joe9663
Copy link

joe9663 commented Apr 6, 2021

Describe the bug
User reported an issue with horizontal scrolling happening on their home and blog pages.

Home page is configured with a left sidebar and set to full-width. Negative margin assignments from the theme framework created a horizontal scroll on mobile/tablet displays, but worked as expected on desktop.

Blog page had a horizontal scroll created by the padding values on the pagination buttons.

To Reproduce
Provided backup of site to James. Issues can be replicated by removing the following CSS from the Customizer in that backup:

.palette-primary .pagination>li>a, .palette-primary .pagination>li>span {
    padding: 0.5em 0.625em;
}
@media (max-width: 991px) {
    .full-width .entry-content .gallery-full-width.tmpl-gallery-7 {
        margin-right: 0;
    }
}
@jamesros161 jamesros161 added bug General bug issues Crio labels May 21, 2021
@jamesros161 jamesros161 added this to the 2.8.1 milestone May 24, 2021
@jamesros161 jamesros161 modified the milestones: 2.8.1, 2.9.1 Jun 30, 2021
@jamesros161 jamesros161 mentioned this issue Oct 11, 2021
@jamesros161 jamesros161 linked a pull request Oct 11, 2021 that will close this issue
@nicolepaschen
Copy link
Member

still happening in RC1. I started with a Wedge Inspiration. Homepage only - inner pages and posts are fine.

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

Successfully merging a pull request may close this issue.

3 participants