Open christian-konrad opened 1 year ago
@pepopowitz given your recent change for images in #3154, do you have any new thoughts on this?
In general, I'm for this change, if we have images that are looking goofy because they're stretched horizontally.
@mesellings do you have thoughts on this?
I agree we should allow some control over image size, as some images shouldn't be full-width (as mentioned/shown here).
The only thing would be to try and keep it within CSS as much as possible, so we can adjust the UX on mobile for larger width images if need be - one approach I've used before is to create a set of CSS classes (image400, image500) etc with pre-defined max-widths, and then people can just add the relevant class onto their image.
Embedding screenshots that do cover only a small fraction of a screen often looks odd, since images auto-scale to the available width. This enlarges the text on the screenshot, which does not harmonize with the article text surrounding it.
We could solve it by allowing to define a per-image max-width, or reduce max-width of the whole article column, or learn something else from other docs with great layouts.
Current:
Proposal: