Skip to content
This repository has been archived by the owner on Jun 26, 2020. It is now read-only.

Fix: The image goes beyond the boundaries of the parent block. Closes… #70

Merged
merged 2 commits into from
Mar 13, 2017

Conversation

oleq
Copy link
Member

@oleq oleq commented Mar 13, 2017

ckeditor/ckeditor5#5073.

Suggested merge commit message (convention)

Fix: The image goes beyond the boundaries of the parent block. Closes ckeditor/ckeditor5#5073.

theme/theme.scss Outdated

// Make sure the image never exceeds the size of the parent container (#67).
max-width: 100%;
max-height: 100%;
Copy link

Choose a reason for hiding this comment

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

Are you sure we need max-height? For me, there is nothing wrong if the long image is longer then editors editable area. Also, it means that a long image will be resized if editable have dynamic height, isn't it?

Copy link
Member Author

Choose a reason for hiding this comment

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

Given the editor's height is not restricted by CSS (parent container), then yes. I wouldn't be so sure there is no such case, though. OTOH, you're probably right.

@pjasiun pjasiun merged this pull request into master Mar 13, 2017
@pjasiun pjasiun deleted the t/67 branch March 13, 2017 13:31
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Side image is not styled correctly
2 participants