clarify another scrolljump bug

matrix-org · Apr 29, 2018 · 731f1fa · 731f1fa
1 parent c254d04
commit 731f1fa
Showing 1 changed file with 2 additions and 0 deletions.
diff --git a/src/components/views/messages/MImageBody.js b/src/components/views/messages/MImageBody.js
@@ -242,6 +242,8 @@ export default class extends React.Component {
         // FIXME: this will break on clientside generated thumbnails (as per e2e rooms)
         // which may well be much smaller than the 800x600 bounding box.
 
+        // FIXME: It will also break really badly for images with broken or missing thumbnails
+
         //console.log("trying to fit image into timelineWidth of " + this.refs.body.offsetWidth + " or " + this.refs.body.clientWidth);
         let thumbHeight = null;
         if (content.info) {