- This topic has 47 replies, 2 voices, and was last updated 6 years, 5 months ago by
Leo.
-
AuthorPosts
-
November 1, 2019 at 11:21 am #1050752
Carrie
Okay, cache is purged, try it now…
https://www.carriemorgan.com/how-my-custom-keyword-research-works/November 1, 2019 at 11:25 am #1050754Leo
StaffCustomer SupportAdd this as well:
.featured-image img { width: 100%; }And since you are using a page builder for content, try the contained option here:
https://docs.generatepress.com/article/page-builder-container/November 1, 2019 at 11:33 am #1050760Carrie
That addition did the trick – thank you so much, Leo. I can’t tell you how much I appreciate your amazing tech support.
I was happy to leave a 5-star review on your WordPress theme page. Thanks again, Leo.
https://wordpress.org/support/topic/fantastic-clean-theme-really-great-tech-support/#post-12088513
November 1, 2019 at 11:35 am #1050761Leo
StaffCustomer SupportThanks so much!
We really appreciate it 🙂
November 1, 2019 at 12:02 pm #1050781Carrie
Just when I thought this was resolved… I’m so sorry to bug you again! The image is resizing great, but the space below it has reappeared.
https://www.carriemorgan.com/how-my-custom-keyword-research-works/
I tried adding that padding code back in to the CSS (below), and it didn’t make it go away. It says something about “don’t use adjoining classes?”
.featured-image.grid-container {
max-width: 100%;
}
.featured-image img {
width: 100%;
}
.site-content { padding-top: 0 ; }It was this originally (before you added the image code):
.site-content { padding-top: 0 !important; }
but when I tried to add that back in underneath your code, it apparently didn’t like the !important. I removed it, but then I get the “don’t use adjoining classes” note and it doesn’t remove the space.November 1, 2019 at 12:10 pm #1050786Leo
StaffCustomer SupportMake sure to see the second part of the my answer here:
https://generatepress.com/forums/topic/hero-image-elements-dont-bring-in-the-alt-tag/page/2/#post-1050754Let me know 🙂
November 1, 2019 at 12:18 pm #1050791Carrie
The screenshot in that article looks like a different version – but I do have this setting, which seems like the closest thing?
Customizing > Layout > Container > One Container
November 1, 2019 at 12:20 pm #1050792Leo
StaffCustomer SupportThat’s a metabox setting so it would be in the Edit page screen.
November 1, 2019 at 12:40 pm #1050801Carrie
Aha! That did the trick. #myhero
November 1, 2019 at 12:50 pm #1050806Leo
StaffCustomer SupportAwesome 🙂
November 1, 2019 at 1:39 pm #1050824Carrie
There’s something about the changes we made that impacts the responsiveness. It strips the side padding on mobile and tablet – so copy bumps right up against the edges of the browser window.
https://www.carriemorgan.com/wp-content/uploads/GeneratePress-screenshot-4.jpg
Can you take a look? Scale the page width down…
November 1, 2019 at 1:59 pm #1050825Leo
StaffCustomer SupportThat’s because of the page builder container option is removing the default theme paddings.
Generally, it’s best practice to let page builder handle the content padding if the content is coming from the page builder so in your case, try adding the desired padding to the visual composer itself 🙂
November 1, 2019 at 2:09 pm #1050829Carrie
Can you please clarify what you mean?
November 1, 2019 at 2:26 pm #1050843Leo
StaffCustomer SupportYou are using Visual Composer to create the content correct?
Try adding the padding to the content using Visual Composer.
November 1, 2019 at 2:30 pm #1050845Carrie
To each content row? That would take hours to do for each page.
I don’t see a way to add side padding in WP Bakery on a site-wide basis.
-
AuthorPosts
- You must be logged in to reply to this topic.