Project

General

Profile

Actions

Patch #26424

closed

Avatar Spacing in Headlines

Added by Felix Gliesche almost 7 years ago. Updated almost 7 years ago.

Status:
Closed
Priority:
Normal
Category:
UI
Target version:
Start date:
Due date:
% Done:

0%

Estimated time:

Description

Currently the space between h2 and an avatar is not correct compared to the space between an avatar and a h3. You can see this problem in the news section. The headline of a news detail view is closer to the avatar than the headline to the avatar on the news index. This is due to a missing space. In addition to this discrepancy I have realized, that there is no need for negative margins for the avatar in h2, and h3 headlines. The only reason I do see for an avatar to have a negative margin within a headline is the case of a h4:

I am attaching a patch series, which a) adds a space between the avatar and the text for h2 headline and b) removes negative margins for avatar images within h2 and h3 headlines. The following images show the difference. Please note, that the difference due to b) is visible in the vertical space to the text after the headline. Because the negative margins are removed the flow is more natural as the baseline is not moved up by some pixels.

before after

Files

h4.jpg (42.5 KB) h4.jpg Felix Gliesche, 2017-07-13 11:41
h3-after.png (42.5 KB) h3-after.png Felix Gliesche, 2017-07-13 11:41
h3-before.png (42.6 KB) h3-before.png Felix Gliesche, 2017-07-13 11:41
h2-after.png (49.9 KB) h2-after.png Felix Gliesche, 2017-07-13 11:41
h2-before.png (49.7 KB) h2-before.png Felix Gliesche, 2017-07-13 11:41
0001-Add-space-between-avatar-and-headline.patch (849 Bytes) 0001-Add-space-between-avatar-and-headline.patch Felix Gliesche, 2017-07-13 11:43
0002-Remove-negative-margin-for-avatar-in-h2-and-h3.patch (992 Bytes) 0002-Remove-negative-margin-for-avatar-in-h2-and-h3.patch Felix Gliesche, 2017-07-13 11:43
Actions #1

Updated by Go MAEDA almost 7 years ago

  • Target version set to 3.4.2
Actions #2

Updated by Jean-Philippe Lang almost 7 years ago

  • Status changed from New to Closed
  • Assignee set to Jean-Philippe Lang

Patches committed, thanks.

Actions

Also available in: Atom PDF