Automattic / default-small-business-theme

GNU General Public License v2.0
10 stars 7 forks source link

Fix Scrolling Issues #9

Closed Aurorum closed 5 years ago

Aurorum commented 5 years ago

There's currently a few scrolling issues with the Business Theme, both on the demo and individual sites. This PR intends to fix those.

On the demo, you'll see that on larger screens, a scrolling bar appears on the bottom. This does not benefit the site, but is just a bit of a nuisance. hfdgdfghdfghgfh

This is also the case with some posts when Related Posts is enabled, and one of those posts has a long title.

gsdfsdfgsfdg

So I've also adjusted related posts so that the scrolling issue should not appear, but this should happen with longer titles instead.

sgdfdgfsdfg

This also should take care of the scrolling issue when a cover image is inserted using Gutenberg. Currently, there is the unnecessary option to scroll at the bottom.

sdgfsdfgsdgsdfg

Would appreciate all feedback on these changes (cc @allancole & @mendezcode) - this is my first PR here and I hope I'm not doing anything wrong, so please let me know if I am!

Aurorum commented 5 years ago

Can no longer recreate.