# Проблема с рендерингом Markdown при изображении, окруженном HTML
**URL:** https://meta.discourse.org/t/markdown-rendering-issue-with-image-surrounded-with-html/152801
**Category:** Bug
**Created:** [26.Май.2020 17:21:21 UTC](https://meta.discourse.org/t/markdown-rendering-issue-with-image-surrounded-with-html/152801 "2020-05-26T17:21:21Z")
**Posts on this page:** 5
**Page:** 2
### Author:  [@angus](https://meta.discourse.org/u/angus)
#### Post date: [11.Август.2021 04:25:10 UTC](https://meta.discourse.org/t/markdown-rendering-issue-with-image-surrounded-with-html/152801/27 "2021-08-11T04:25:10Z")
Да, если вы полностью пойдете по этому пути, вам придется пожертвовать возможностью редактирования контента в Discourse ради полной целостности HTML.
Мне казалось, что может существовать некий компромисс, при котором этот подход применяется только к тегам `img` в импортированных HTML-постах.
Но сейчас я начинаю сомневаться в этом. Выделение этого подхода потребует изменений во многих частях обработки постов.
---
### Author:  [@david](https://meta.discourse.org/u/david)
#### Post date: [11.Август.2021 11:29:48 UTC](https://meta.discourse.org/t/markdown-rendering-issue-with-image-surrounded-with-html/152801/28 "2021-08-11T11:29:48Z")
> [@angus](#):
>
> Да, если говорить проще, возможно, код мог бы поддерживать загрузку горячих ссылок на изображения без необходимости преобразования `img` в markdown.
Да, я думаю, что это, вероятно, лучший вариант. Я начал работу над этим ещё в июне 2020 года, но в итоге это потребовало слишком много усилий, и мне пришлось переключиться на другие проекты. У меня было несколько подходов к разрешению URL-адресов вида `upload://` в тегах `
>
> This is implemented within the markdown.it pipeline. Each 'html\_block' token is …scanned for img nodes with upload-protocol src attributes. These are then translated into real URLs using the existing method.
Этот вариант можно улучшить, используя на сервере стандартную реализацию DOM на JavaScript (например, jsdom), но это кажется довольно громоздким решением.
### Вариант 2 реализации:
Разрешить атрибуты src вида `upload://` на всём протяжении конвейера markdown, а затем заменить их позже. На клиенте это на самом деле довольно просто — мы уже асинхронно заменяли URL-адреса `upload://` после обработки. На сервере это требует дополнительного шага обработки с использованием Nokogiri.
Плюсы: парсер соответствует стандартам HTML5
Минусы: различная реализация на клиенте и сервере, немного усложняет конвейер
>
>
> This is implemented outside the markdown pipeline. The upload:// src is left in …place during markdown cooking, then replaced afterwards. When cooking in the browser, this is handled in a decorateCooked method. On the server, it is handled using Nokogiri immediately after cooking.
* * *
Я думаю, что вариант 2, вероятно, стоит выбрать. Тогда нам нужно будет обновить задачу `pull_hotlinked_images`, чтобы она сохраняла теги `
### Author:  [@mattdm](https://meta.discourse.org/u/mattdm)
#### Post date: [20.Август.2021 21:06:55 UTC](https://meta.discourse.org/t/markdown-rendering-issue-with-image-surrounded-with-html/152801/29 "2021-08-20T21:06:55Z")
Я действительно не понимаю, в чём здесь сложность. Очевидно, что HTML-тег изображения заменяется на markdown — например, ``. Почему бы просто не добавить два символа переноса строки перед `!`? Это обеспечит корректный рендеринг и позволит функции загрузки изображений работать, предотвращая битые картинки и проблемы с межсайтовыми запросами.
Существует ли реальная, а не теоретическая ситуация, когда этот пробел может вызвать проблему? И является ли эта проблема _хуже_, чем текущее состояние плагина, когда «картинки постоянно ломаются»?
@david, вы отмечаете, что «решение с переносом строки, вероятно, не будет реализовано», потому что «для нас ключевым моментом является сохранение целостности контента». Но целостность контента уже нарушена при вставке тегов изначально. Я… действительно не понимаю, как это может быть лучше.
В данный момент _каждый раз_, когда кто-то добавляет изображение в свой пост в WordPress, картинки возвращаются битыми, и мне часто приходится сталкиваться с комментариями «картинки не работают», которые теперь часто сопровождаются ответами «да, это потому что Discourse ужасен». Я хотел бы избежать обоих этих проблем.
Я понимаю, что настройка «не загружать изображения» может служить обходным путём, но на самом деле я _хочу_, чтобы изображения загружались, поэтому надеюсь, что это можно будет использовать лишь как временное решение.
---
### Author:  [@david](https://meta.discourse.org/u/david)
#### Post date: [29.Март.2022 10:51:21 UTC](https://meta.discourse.org/t/markdown-rendering-issue-with-image-surrounded-with-html/152801/30 "2022-03-29T10:51:21Z")
Это должно быть исправлено следующим образом:
>
>
> Previously, our \`upload://\` protocol urls were only supported in markdown image …tags. This meant that our PullHotlinkedImages job was forced to convert \`\
> This commit adds support for \`upload://\` inside regular HTML \`\
> Context at https://meta.discourse.org/t/152801
>
>
> Under some conditions, replacing an \`\
> Context at https://meta.discourse.org/t/152801
Давайте попробуем это здесь на Мета.
```plaintext
Изображение в HTML-блоке:
### Author:  [@david](https://meta.discourse.org/u/david)
#### Post date: [04.Апрель.2022 07:00:57 UTC](https://meta.discourse.org/t/markdown-rendering-issue-with-image-surrounded-with-html/152801/31 "2022-04-04T07:00:57Z")
Эта тема была автоматически закрыта через 5 дней. Новые ответы больше не принимаются.
[Previous page](https://meta.discourse.org/t/markdown-rendering-issue-with-image-surrounded-with-html/152801.md?page=1)