Chainlit / chainlit

Build Conversational AI in minutes ⚡️
https://docs.chainlit.io
Apache License 2.0
7.24k stars 955 forks source link

Improve UX when loading images #1488

Closed fgalind1 closed 3 weeks ago

fgalind1 commented 3 weeks ago

Backport https://github.com/Chainlit/chainlit/pull/1402 already merged from main/development branch 2.X into releases/1.3


Currently when adding images in the conversation, if the image is big and is not that clear, clicking in the image will attempt to download the image.

This PR attempts to improve the UX a bit by just opening the image in a new window (aka popup) to see the image larger. It has a download option on this new popup as well to preserve the original download behavior, but now the UX is a bit better

dokterbob commented 3 weeks ago

Thanks but... We're doing semver now, so the next release adding features would be 1.4 -- but we might just jump right to 2.0 (as we've limited capacity to maintain older code). 1.3 will receive only fixups.

(You're welcome to run 2.0.dev2 though, or even run directly from main, gives us lot's of extra testing!)

fgalind1 commented 3 weeks ago

Thanks but... We're doing semver now, so the next release adding features would be 1.4 -- but we might just jump right to 2.0 (as we've limited capacity to maintain older code). 1.3 will receive only fixups.

(You're welcome to run 2.0.dev2 though, or even run directly from main, gives us lot's of extra testing!)

sounds good, I'll test it using 2.0.dev2 - thanks @dokterbob