Skip to content

refactor: clearing floats in a modern way#1362

Merged
JosefBredereck merged 1 commit intopattern-lab:devfrom
mfranzke:refactor/clearing-floats-in-a-modern-way
Oct 16, 2021
Merged

refactor: clearing floats in a modern way#1362
JosefBredereck merged 1 commit intopattern-lab:devfrom
mfranzke:refactor/clearing-floats-in-a-modern-way

Conversation

@mfranzke
Copy link
Contributor

@mfranzke mfranzke commented Oct 14, 2021

The previous solution with overflow(-x) leads to several problems, previously regarding overflow on absolutely positioned elements being cropped, and now elements that are positioned sticky wouldn't work because of the overflow-x declaration.

Closes #1361

Summary of changes:

As we don't want to actually control any overflow here, but mainly clear the remaining defined floats by the components, it might be much better to switch to an actual clearfix.

As the previous solution with overflow(-x) leads to several problems
@mfranzke mfranzke requested a review from sghoweri as a code owner October 14, 2021 11:19
@JosefBredereck JosefBredereck merged commit c8ecb98 into pattern-lab:dev Oct 16, 2021
@mfranzke mfranzke deleted the refactor/clearing-floats-in-a-modern-way branch October 16, 2021 14:53
antonia-rose pushed a commit to quelltexterin/nemo-uikit-workshop that referenced this pull request Apr 12, 2023
As the previous solution with overflow(-x) leads to several problems
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Legacy floating clearfix solution prevents sticky positioning

2 participants