Improve behavior of image columns

[Janpot]

• Don't show anything if there is no value
• Show a thumbnail with a popover, avoid auto row height

Screen.Recording.2023-02-07.at.12.01.03.mov

[apedroferreira]

Is there a way to have more control over the image size, or should there?
Anyway this solution works for me in that sense as not sure what the best approach would be, could depend on different use cases...

Found a bug though, on the 2nd item the popover was looking really big to me and had a scroll bar:

Screen.Recording.2023-02-07.at.17.54.01.mov

[apedroferreira]

Just the bug I found, code seems good!

[Janpot]

Is there a way to have more control over the image size, or should there?

You can always create a custom component. I believe this is a more sensible default as it just behaves the way text behaves, i.e. just take the available space. The previous solution uses getRowHeight: () => 'auto', which had the side effect that it resized the rows on other columns as well, which is not the intended behavior.

Found a bug though, on the 2nd item the popover was looking really big to me and had a scroll bar:

I limited the size of the popover image. I also made sure cell alignment still works

[apedroferreira]

You can always create a custom component. I believe this is a more sensible default as it just behaves the way text behaves, i.e. just take the available space.

Sounds good, that's better now that we have custom component columns.
This looks good now!