Skip to content

change(modal): allows image description flexbox to take available space #3269

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

Conversation

ko2in
Copy link
Member

@ko2in ko2in commented May 12, 2025

Description

The flex item for the image description in the modal does not utilize the available space, which restricts its child elements from being displayed at full width when required.

It results in considerable adverse layout impacts when an element, such as a form is displayed inside the description container. The form components, including input fields, fail to utilize the available space, causing the form to appear constricted within the modal.

This PR modifies this behavior of the flexbox item for the description, enabling it to be flexible and allowing it to grow and shrink.

So, the description container will grow in the absence of sibling elements following it, allowing the child elements within to take available space and display in a full-width format, particularly in the context of the form layout.

Testcase

Before: https://jsfiddle.net/f9uwm0ov/

After: https://jsfiddle.net/ko2in/d7o8fjak/

Screenshot

Before

Before

After

After

Closes

#3268

The flex item of image description in modal doesn't take available space
that limits it's child elements from displaying full-width when necessary.
Copy link
Member

@lubber-de lubber-de 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 added type/bug Any issue which is a bug or PR which fixes a bug lang/css Anything involving CSS labels May 15, 2025
@lubber-de lubber-de added this to the 2.10.0 milestone May 15, 2025
@lubber-de lubber-de merged commit 528c1b4 into fomantic:develop May 15, 2025
8 checks passed
@ko2in ko2in deleted the change-modal-image-description-flexbox-state branch May 19, 2025 09:58
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.

2 participants