Skip to content

[Grid] stackable grid had wrong width inside popup #1739

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Nov 2, 2020

Conversation

lubber-de
Copy link
Member

Description

A stackable grid inside a popup did not have enough specificity to adjust the width on mobile views

Testcase

Screenshot

gridinsidepopup

@lubber-de lubber-de added type/bug Any issue which is a bug or PR which fixes a bug lang/css Anything involving CSS state/awaiting-reviews Pull requests which are waiting for reviews labels Nov 1, 2020
@lubber-de lubber-de added this to the 2.8.8 milestone Nov 1, 2020
Copy link
Contributor

@exoego exoego left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@lubber-de lubber-de merged commit 79b4eff into fomantic:develop Nov 2, 2020
@lubber-de lubber-de deleted the fix/stackableGridInPopup branch November 2, 2020 08:45
@lubber-de lubber-de removed the state/awaiting-reviews Pull requests which are waiting for reviews label Nov 2, 2020
lubber-de added a commit that referenced this pull request Jan 8, 2023
This PR reverts and corrects #1739 for stackable nested grids.
The margins on mobile screens for stackable grid has now been adjusted so it only takes place when a stackable grid is not nested in another grid or a segment.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
lang/css Anything involving CSS type/bug Any issue which is a bug or PR which fixes a bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants