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

Extra space added when using a feature_row #2327

Closed
lsolesen opened this issue Nov 19, 2019 · 5 comments
Closed

Extra space added when using a feature_row #2327

lsolesen opened this issue Nov 19, 2019 · 5 comments

Comments

@lsolesen
Copy link
Contributor

lsolesen commented Nov 19, 2019

Extra space added when using a feature_row in layout single.

Environment

Expected behavior

There should be no extra space in front of the feature row.

Steps to reproduce the behavior

See https://github.com/motionsplan/motionsplan-dk-jekyll/blob/gh-pages/_posts/2019-11-19-regler-styrkeloft.md

Which gives this result:

Screenshot - 2019-11-19T083854 829

Other

It can be seen on the live site here: https://www.motionsplan.dk/powerlifting-rules/

@mmistakes
Copy link
Owner

Looks like it has to do with the right sidebar. If you inspect it you can see it's pushing over into the main content column some, which pushes the feature row down.

image

When the right sidebar isn't being floated the feature row has enough room and fits.

image

Not sure what the fix is yet.

@iBug
Copy link
Collaborator

iBug commented Nov 19, 2019

Duplicate of #2263

The answer is, if you want to use that in single layout, you should look into gallery instead. feature_row is intended for splash layout.

@mmistakes
Copy link
Owner

It'll likely break other things but giving feature__wrapper an inline style of display: inline-block fixes the wrap with the sidebar.

image

@iBug
Copy link
Collaborator

iBug commented Nov 19, 2019

@mmistakes Perhaps something like this would be helpful without breaking anything?

.layout--single .feature__wrapper {
  display: inline-block;
}

Tried this in a few MM-themed sites, didn't notice any failure so far.

@mmistakes
Copy link
Owner

@iBug that change looks harmless to me if you want to submit a PR with the fix.

iBug added a commit to iBug/minimal-mistakes that referenced this issue Nov 21, 2019
kamil-adam pushed a commit to twocolumn/minimal-mistakes that referenced this issue Apr 1, 2020
jesuswasrasta pushed a commit to jesuswasrasta/jesuswasrasta.github.io that referenced this issue Jul 8, 2020
kaitokikuchi pushed a commit to kaitokikuchi/kaitokikuchi.github.io that referenced this issue Sep 4, 2023
chukycheese pushed a commit to chukycheese/chukycheese.github.io that referenced this issue Sep 18, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants