Twenty Fourteen: Gap beneath header image

[taupecat]

When user adds a header image (see #25400), a gap appears between the image and the masthead. Changing the header image's display type to "block" fixes.

[taupecat]

Screenshot of gap appearing between header image and masthead

[taupecat]

Patch of style.css to fix the gap by applying "display: block;" to the header image

[ramiabraham]

Same on my nightly, good catch.

[obenland]

I think we should use the a img selector in line 84 for that since this is a problem with all linked images. It would also make the selector in line 1509 unnecessary.

iamtakashi, can you confirm this?

[taupecat]

Changing the a img {} selector makes sense. I'll change my patch accordingly.

However, I'm not seeing how the selector at 1509 is affected...

.hentry .mejs-controls .mejs-time-rail .mejs-time-total,
.hentry .mejs-controls .mejs-horizontal-volume-slider .mejs-horizontal-volume-total {
	background: #595959;
}

[iamtakashi]

Replying to obenland:

I think we should use the a img selector in line 84 for that since this is a problem with all linked images. It would also make the selector in line 1509 unnecessary.

iamtakashi, can you confirm this?

I think you're correct. This will be an issue for all linked image so it makes sense to use the selector, a img. And we should be able to get rid of vertical-align: middle; from line 1383 - 1389 (I think this is what you meant).

[obenland]

Replying to iamtakashi:

I think you're correct. This will be an issue for all linked image so it makes sense to use the selector, a img. And we should be able to get rid of vertical-align: middle; from line 1383 - 1389 (I think this is what you meant).

Cool. Right, ​line 1383. I hadn't svn up'ed in a while ;)

[taupecat]

This patch instead of 25404.diff; puts "display: block" on all linked images and removes deprecated rules

[lancewillett]

In 25611:

Twenty Fourteen: fix gap beneath header image, props taupecat. Fixes #25404.