Twenty Fifteen: links display in RTL shouldn't be inline-block"

[yoavf]

Not sure if this is a leftover or intentional, but the following rule from rtl.css is problematic:

a {
	display: inline-block;
}

It causes a vertical bump every time a link is hovered on, because of the bottom border:
​https://cloudup.com/cCTdfSN46WD

[lancewillett]

Patch looks good to me.

[iamtakashi]

Without the rule, how can we deal with these stuff? I'd like to know better solution for this.

Without:

With:

I'm aware of I'm not testing with real RTL language though.

[yoavf]

Replying to iamtakashi:

I'm aware of I'm not testing with real RTL language though.

You should - things will just sort themselves :)

[iandstewart]

The positioning of the icons looks off to me with the patch applied.

[iamtakashi]

Replying to yoavf:

You should - things will just sort themselves :)

Will it really? I'm talking about icons position and post count in the widget.

[yoavf]

Replying to iamtakashi:

Will it really? I'm talking about icons position and post count in the widget.

Yes. Here's ltr and rtl with the patch applied, with content in Hebrew:

And here are the widgets with the counts:

[iamtakashi]

Oh wow, this is good to know. I really should test with a real RTL language.

If we don't need to have the style, we can remove some more styles from rtl.css. I'll work on a refresh patch.

[iamtakashi]

I'm having a trouble to test with a real RTL language but here is the update patch that removes some of unnecessary styles.

[iandstewart]

In 30399:

Twenty Fifteen: Removing unneeded CSS from RTL causing odd hover effects.

Props yoavf, iamtakashi, fixes #30360.