Изменение размера изображения, похоже, не работает

На Discourse 2.3.0.beta3 Release Notes сообщается:

Я заметил, что в последнее время большие изображения отображаются на всю ширину, а высота обрезается.
Например:

Разметка для этого изображения выглядит так:
![Screenshot_20200321-114212|210x500,200x](upload://r0GmIGqITPIix8sJgRqTXRNhuHA.jpeg)

Я также пробовал
![Screenshot_20200321-114212|210x500,25%](upload://r0GmIGqITPIix8sJgRqTXRNhuHA.jpeg)
но результат тот же — изображение растягивается на всю ширину.

При редактировании в окне предварительного просмотра изображение отображается корректно:

2 лайка

Да, функция обрезки высоких изображений у нас очень запутанная.

У меня есть предложенное исправление здесь.

@vinothkannans, пожалуйста, примени его, убедись, что тесты проходят, и приведи здесь несколько примеров?

10 лайков

Означает ли это, что я что-то не понимаю и делаю неправильно? Я вовсе не хочу звучать оборонительно, просто если дело лишь в том, что всё запутанно и я чего-то не уловил, я был бы рад узнать, что именно мне нужно исправить.

5 лайков

Большие изображения обрезаются в соответствии с соотношением сторон загружаемого изображения (существует настройка min ratio to crop, которая определяет, когда выполняется обрезка)… эта обрезка применяется даже в том случае, если размеры в формате Markdown меньше порога обрезки.

Таким образом, вы ничего не делаете неправильно, кроме как не знаете, как работает порог min ratio to crop (было бы немного безумно ожидать, что каждый поймёт это, просто взглянув на изображение, которое собирается загрузить, а настройка соотношения сторон непонятна обычным пользователям!).

Вносимое исправление обеспечит учёт размеров в формате Markdown перед выполнением обрезки.

8 лайков

Ничего плохого не делаю. Просто представьте: пользователь загрузил изображение размером 3 пикселя в ширину и 50 000 пикселей в высоту. Как бы выглядела тема, если бы это изображение отображалось с сохранением пропорций? А что, если пользователь будет публиковать это изображение снова и снова? :thinking:

4 лайка

Это напомнило мне: я давно хотел упомянуть, что очень высокие и узкие изображения действительно ломают Lightbox:

fiddy2|0x500

Эти размеры это отлично демонстрируют :wink:

3 лайка

Правильно, но представьте это без кадрирования по высоте!

4 лайка

Спасибо! Вы отлично передали суть «непонимания»! :smile:

4 лайка

Я думаю, что @nbianca исправил эту проблему, и исправление уже развернуто!

5 лайков

Да, я объединил этот PR два месяца назад, поэтому сейчас всё должно работать корректно. :blush:

5 лайков

Ура, и спасибо! Теперь в нашем Сообществе можно публиковать красивые полноэкранные скриншоты со смартфонов!

5 лайков

Эта тема была автоматически закрыта через 4 дня. Новые ответы больше не принимаются.