Archived topic
Hero image elements don't bring in the Alt tag
47 replies · Started by Carrie on October 31, 2019
Okay, cache is purged, try it now...
https://www.carriemorgan.com/how-my-custom-keyword-research-works/
Add 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/
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
Thanks so much!
We really appreciate it :)
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.
Make 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-1050754
Let me know :)
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
That's a metabox setting so it would be in the Edit page screen.
Aha! That did the trick. #myhero
Awesome :)
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...
That'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 :)
Can you please clarify what you mean?
You are using Visual Composer to create the content correct?
Try adding the padding to the content using Visual Composer.
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.