1
0
Fork 1

Remove whitespace above headline #13

Open
opened 1 year ago by SomeGuy · 2 comments
SomeGuy commented 1 year ago
Owner

image

This can probably fixed with some float value.

![image](/attachments/d4e8f308-6b93-4143-a6d2-6b7825fb7169) This can probably fixed with some `float` value.
gravel commented 1 year ago
Collaborator

TLDR: This works as intended.

Floated elements shrink the available space for centering, and this layout would pose other issues even if the boxes shared the same space (such as with absolute positioning).

To add to that, in a successful implementation of the above idea, the headline would be missing an upper margin that would have to be added again, with the only difference being that the margin is now visually symmetric.

Note: A floated element followed by a clear: both; separator would produce exactly the same layout as we have. Float is good for images floating in text. Not much else nowadays.

TLDR: This works as intended. Floated elements shrink the available space for centering, and this layout would pose other issues even if the boxes shared the same space (such as with absolute positioning). To add to that, in a successful implementation of the above idea, the headline would be missing an upper margin that would have to be added again, with the only difference being that the margin is now visually symmetric. Note: A floated element followed by a `clear: both;` separator would produce exactly the same layout as we have. Float is good for images floating in text. Not much else nowadays.
gravel commented 1 year ago
Collaborator

Requesting close as wontfix. @SomeGuy

Requesting close as `wontfix`. @SomeGuy
This repo is archived. You cannot comment on issues.
No Label
No Milestone
No Assignees
2 Participants
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: SomeGuy/sessioncommunities.online#13
Loading…
There is no content yet.