proper aligning for tabs

[diomed]

This is a bugfix.

Proposed solution

[x] fix for #2172

replaces center with baseline, so all tabs are properly aligned.

[jgthms]

Do you have an example where this is an issue?

[diomed]

example is shown in issue which I'm closing.
problem is explained here

[jgthms]

The solution for that particular problem is to change the following rule in content.sass:

li + li
  margin-top: 0.25em

And make sure it's not applied for tabs.

[diomed]

so do I change to 0 and modify PR?
is this it?

[jgthms]

Yeah change this PR so this selector li + li doesn't target li that are within a .tabs.

[diomed]

didn't I get it?
is this good in the end or not?
I'm unsure why it's not merged

[jgthms]

No that’s the wrong output. The “not tabs” part should be for the ul, not the li.

[diomed]

@jgthms so - like this then?

if not the case, can you please update it properly so this one could be fixed already.

[jgthms]

Just add before // Sizes:

.tabs
  li + li
    margin-top: 0

[jgthms]

You've indented that one level too far. And you can remove the rest.

[diomed]

like this?