Skip to content
This repository has been archived by the owner on Mar 13, 2024. It is now read-only.

MM-19909 Fix for scroll pop with markdown images #4164

Merged
merged 1 commit into from
Nov 11, 2019
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion components/__snapshots__/size_aware_image.test.jsx.snap
Original file line number Diff line number Diff line change
Expand Up @@ -29,10 +29,10 @@ exports[`components/SizeAwareImage should render a placeholder and has loader wh
Object {
"maxHeight": 200,
"maxWidth": 300,
"verticalAlign": "middle",
}
}
viewBox="0 0 300 200"
width="inherit"
xmlns="http:https://www.w3.org/2000/svg"
/>
</div>
Expand Down
3 changes: 2 additions & 1 deletion components/size_aware_image.jsx
Original file line number Diff line number Diff line change
Expand Up @@ -133,7 +133,8 @@ export default class SizeAwareImage extends React.PureComponent {
<svg
xmlns='http:https://www.w3.org/2000/svg'
viewBox={`0 0 ${dimensions.width} ${dimensions.height}`}
style={{verticalAlign: 'middle', maxHeight: dimensions.height, maxWidth: dimensions.width}}
style={{maxHeight: dimensions.height, maxWidth: dimensions.width}}
width='inherit'
/>
</div>
);
Expand Down
2 changes: 1 addition & 1 deletion sass/layout/_markdown.scss
Original file line number Diff line number Diff line change
Expand Up @@ -59,7 +59,7 @@ h6 {
max-height: 500px;
}

div .markdown-inline-img {
div.markdown-inline-img {
min-width: 50px;
display: inline-block;
width: auto;
Expand Down