search

Odonno / modern-gitter-winjs

Windows 10 client of gitter, written in WinJS
https://www.microsoft.com/store/apps/modern-gitter/9wzdncrd2tzj
9 stars 0 forks source link

Problem with images in room #37

Closed gep13 closed 8 years ago

gep13 commented 8 years ago

I noticed an issue in a couple rooms when using the application this morning.

When I entered each room, there were images posted into some of the messages, and the text from other messages were over the top of the images. I was hoping to get a screenshot of the problem, but when I got to my computer and opened the rooms again, the images are appearing correctly.

If it happens again, I will update this issue with a screenshot.

Odonno commented 8 years ago

I think I had the same and maybe caused by the same bug that happens in #35.

Once the UI is refreshed (ex: scroll), the message height is recalculated based on image height. There is also another bug if the image is too big (it goes beyond screen width).

capture

Odonno commented 8 years ago

I just got the bug you describe right now on my other device.

capture

Indeed, refreshing the UI should fix the bug.

gep13 commented 8 years ago

Yip, that last screenshot is exactly what I was seeing on my phone as well.

Odonno commented 8 years ago

@gep13 The new version (2.3.6) should have fixed this bug.