Skip to content
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

[Modal] Broken "image content" layout #599

Closed
donaggio opened this issue Mar 29, 2019 · 3 comments
Closed

[Modal] Broken "image content" layout #599

donaggio opened this issue Mar 29, 2019 · 3 comments
Assignees
Labels
lang/css Anything involving CSS type/bug Any issue which is a bug or PR which fixes a bug
Milestone

Comments

@donaggio
Copy link

Bug Report

Modal with "image content" has broken layout: textual content flows under the image instead of staying at its right

Steps to reproduce

  1. https://fomantic-ui.com/modules/modal.html#scrolling-modal
  2. Click on Scrolling Modal or Internally Scrolling Content "Run Code" button

Expected result

  1. https://semantic-ui.com/modules/modal.html#scrolling-modal
  2. Click on Scrolling Modal or Internally Scrolling Content "Run Code" button

Version

2.7.2

@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 Mar 29, 2019
@prudho
Copy link
Contributor

prudho commented Mar 29, 2019

Introduced by 6f4f09c. Removing the line flex-wrap: wrap; seems to fix this issue.

@lubber-de lubber-de self-assigned this Mar 29, 2019
@lubber-de
Copy link
Member

@prudho Yes, but then #209 will not work anymore. 😢
I'll investigate

@lubber-de
Copy link
Member

lubber-de commented Mar 29, 2019

Fixed by #600
See https://jsfiddle.net/kdvum7ho/2/

@prudho You were right, in the end i had to remove the flex-wrap.

@lubber-de lubber-de modified the milestones: 2.7.4, 2.7.5 Mar 29, 2019
@lubber-de lubber-de added the tag/next-release/nightly Any issue which has a corresponding PR which has been merged and is available in the nightly build label Apr 2, 2019
@lubber-de lubber-de removed the tag/next-release/nightly Any issue which has a corresponding PR which has been merged and is available in the nightly build label May 16, 2019
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

No branches or pull requests

3 participants