Images are not centralized or are breaking the layout

[MatheusFD]

When you upload an image in a ticket, this image get a permalink, like this https://meta.trac.wordpress.org/attachment/ticket/564/banner-1544x500-no-gender.png and https://meta.trac.wordpress.org/attachment/ticket/564/banner-772x250-no-gender.png

The problem is: when it's a big image, it's breaking the layout and when it's a small image, it's aligned to the left. I think that the best thing here is put it in the center.

To understand better, I'll upload some images too

[Otto42]

I kind of disagree with this. It's an attachment. It's supposed to show the actual attachment, not a resized version of it. And centering it doesn't help with understanding things like the width of the image.

Since the attachment displays resized properly in the ticket itself, then I kind of don't care what the attachment page looks like, insofar as I want to see the details of that attachment as it actually is, not some presentation changed version of it.

https://meta.trac.wordpress.org/ticket/564

[MatheusFD]

The problem is when it breaks the layout, I think. I think it makes the page a little bit ugly :/

[Otto42]

I agree that it makes the page ugly, but you don't look at the direct attachment page for pretty. You look at it to get the file or to understand the problem the attachment is solving or something else. Making things prettier at the expense of having actual information to work with immediately visible to you isn't the best way forward either.

-1 to centering, because this hides valuable information (width)

+0 on scaling down large images, leaning towards against.

[MatheusFD]

What do you think? In let the page in this way?

[MatheusFD]

about centering images, if you look, it's a small image, that don't cover all the area. I'm not changing its width :/

[Otto42]

No, you're not changing the width, but by centering it, you make it not start at the left hand side anymore, so the width of the image is not immediately visible and comparable to other images on similar pages.

I can't compare different images in different tabs easily because their left-side starting points could now be different. They're located at different places on the page.

[MatheusFD]

Oh, ok! I disagree, because I think layout not breaking is very important. If you click any page to compare, the minimum I wait is everything in the right place. If you enter in the page and it's with a broken image, how will you compare? But ok :/

[SergeyBiryukov]

I tend to agree with max-width: 100%, but disagree with text-align: center, for the same reason as in comment:6.

[MatheusFD]

@SergeyBiryukov ok! So, in the second patch, it's correct! Thanks for the usability class you two (@Otto42) hahaha

[SergeyBiryukov]

On second thought, looks like Trac doesn't have a link to view images in full size without downloading, so the way it works now is the only way to get an accurate preview for large attachments, even if the page doesn't look pretty.

[MatheusFD]

ok. So let's maintain the page the way it is. Thanks for the reports!

[ocean90]

#5297 was marked as a duplicate.