Lukas Masuch

Can you also do one update for the `date-popover-sidebar.snap.png` e2e screenshot? The font changes might not fall under the threshold, but it is better to push with the most up-to-date...

> so that this change is fully verified on the CI. Is the license check also running inside the CI? I thought it is only something that is added/updated when...

I tried to look through all files if there is anything where we shouldn't add our copyright, and besides the `openmetrics_data_model.proto` file I think we are fine. But maybe it...

I also did a quick check today into this issue. The duration type is only partially implemented in the javascript/typescript version of Arrow. For example, it is missing here: https://github.com/apache/arrow/blob/f5166fe21969d19adff23fc840ed1d7511348bad/js/src/ipc/metadata/message.ts#L440...

@wizzioficial Thanks for reporting this! `st.columns` are built to be responsive, meaning that it will switch to rendering the columns underneath each other at a certain display size. So, this...

I think we need some more tests for this as well. Maybe even some more e2e tests to test different SVGs.

I'm not sure if I can give a reasonable review since I don't really have a lot of experience with SVGs yet. From doing a quick research, the sizing/scaling of...

If I understand it correctly, there are the following scenarios: 1) SVG with viewbox and width/height properties 2) SVG with viewbox but no width/height properties 3) SVG without viewbox but...

Oki, we need to keep `markup` for now to support scenario 4 (or if it contains external content). However, I think it might be reasonable to somehow deprecate this support...

[sg-image-svg-sizing-7.snap.png](https://github.com/streamlit/streamlit/pull/5189/files#diff-9c36f95a899485dbccaf098f670b77329ec5b89c9e55c055d59e9925fd40da3f) can be delete, or? I don't see this snapshot referenced in the code.