Markdownのプレビューと結果が異なります

rokejulianlockhart · 2023 年 5 月 4 日午後 8:05

Canapin · 2023 年 5 月 4 日午後 8:22

Reproできました。プレビューと投稿が確かに異なります。
タグの上下の空行はここではあまり関係ありませんが、重要ではありません。それらを削除すると、最も近い次のコンテンツと前のコンテンツに折り返されますが、すべてのコンテンツに折り返されるわけではありません。

空行なし

さらに、なぜ

import QtQuick
import QtQuick.Controls 2.15 as QQC2
import QtQuick.Layouts
import QtPositioning

そして Action, Button, ApplicationWindow, Frame の前に QQC2 を追加することが機能せず、

import QtQuick
import QtQuick.Controls
import QtQuick.Layouts
import QtPositioning
import QtQuick.Controls 2.15 as QQC2

そして QQC2.Button の追加（[qml+pyqt6] Can't get native control appearance - #2 by carl - Help - KDE Discuss より）は機能するのでしょうか？
新しい行

![image|690x412](upload://rIR9RPpxTxaXhuJmjQRWoOBO6ie.png)

とはいえ…
問題は <strike>^[1] がインライン要素だったからだと思っていました。 や  のような他のタグでも再現できますが、 や  では再現できません。したがって、インラインレベルの要素だからというわけではありません。私の見たところ、 や  では、Discourse はブロックをインラインでラップすることを許可しています。

<strong>

さらに、なぜ

```qml
import QtQuick
import QtQuick.Controls 2.15 as QQC2
import QtQuick.Layouts
import QtPositioning
```

そして `Action`, `Button`, `ApplicationWindow`, `Frame` の前に `QQC2` を追加することが機能せず、

```qml
import QtQuick
import QtQuick.Controls
import QtQuick.Layouts
import QtPositioning
import QtQuick.Controls 2.15 as QQC2
```

そして `QQC2.Button` の追加（https://discuss.kde.org/t/qml-pyqt6-cant-get-native-control-appearance/1240/2?u=rokejulianlockhart より）は機能するのでしょうか？
新しい行

</strong>

これは承認済みタグとは関係ないようです。Discourse はこれらのタグ（<s>, , , ,  など）をすべて受け入れているからです。

これは markdown-it の癖でもないようです。彼らのデモでは問題を再現できませんでした。

<s> の代わりに <strike> が非推奨であることに注意してください。この問題には影響しませんが、知っておくと良いでしょう…私も今知りました ↩︎

Arkshine · 2023 年 5 月 5 日午前 1:28

好奇心でざっと見てみました。Nokogiri ライブラリの問題のようです。

Discourse がここで使用しているものによると:

github.com/discourse/discourse

lib/pretty_text.rb

7ff8e5580


      
            doc = Nokogiri::HTML5.fragment(sanitized)
          
            add_nofollow = !options[:omit_nofollow] && SiteSetting.add_rel_nofollow_to_user_content
            add_rel_attributes_to_user_content(doc, add_nofollow)
            strip_hidden_unicode_bidirectional_characters(doc)
            sanitize_hotlinked_media(doc)
          
            add_mentions(doc, user_id: opts[:user_id]) if SiteSetting.enable_mentions
          
            scrubber = Loofah::Scrubber.new { |node| node.remove if node.name == "script" }
            loofah_fragment = Loofah.fragment(doc.to_html)
            loofah_fragment.scrub!(scrubber).to_html
          end
          
          def self.strip_hidden_unicode_bidirectional_characters(doc)
            return if !DANGEROUS_BIDI_REGEXP.match?(doc.content)
          
            doc
              .css("code,pre")
              .each do |code_tag|
                next if !DANGEROUS_BIDI_REGEXP.match?(code_tag.content)

Loofah.fragment は Nokogiri の HTML4 パーサーを使用します。

これは Loofah.html5_fragment を使用して修正できる可能性があります。ただし、Nokogiri >= 1.14.0 および Loofah >= 2.21.0 である必要があります。Discourse はすでに Nokogiri::HTML5.fragment を使用しています。これは理にかなっています。

注: Loofah 2.21.0 はまだリリースされていません。現在は RC1 です。

sam · 2023 年 5 月 5 日午前 3:45

素晴らしいデバッグでした！ありがとうございます

sam · 2023 年 5 月 8 日午前 3:30

30日後にブックマークしておきます。新しいルーファが出たら、アップグレードできることを願っています。

sam · 2023 年 5 月 12 日午前 2:36

loofah の HTML5 版を試したところ、セキュリティの問題が発生しているようです。

github.com/discourse/discourse

DEV: use HTML5 version of loofah

main ← loofah

opened 02:35AM - 12 May 23 UTC

SamSaffron

+123 -115

https://meta.discourse.org/t/markdown-preview-and-result-differ/263878 The re…sult of this markdown had different results in the composer preview and the post. This is solved by updating Loofah to the latest version and using html5 fragments like our user had reported. While the change was only needed in [cooked_post_processor.rb](https://github.com/discourse/discourse/pull/21500/files#diff-67de7f44aa04f02ceba9770e5d83b4465add3bd4297be871f94a2233cd4831a7), I've updated the other areas of our codebase to also use the html5 fragment. > <strike> > > Additionally, how come > > ```qml > import QtQuick > import QtQuick.Controls 2.15 as QQC2 > import QtQuick.Layouts > import QtPositioning > ``` > > and prepension of `QQC2` before `Action`, `Button`, `ApplicationWindow`, and `Frame` doesn't work, whereas > > ```qml > import QtQuick > import QtQuick.Controls > import QtQuick.Layouts > import QtPositioning > import QtQuick.Controls 2.15 as QQC2 > ``` > > and addition of `QQC2.Button` (per https://discuss.kde.org/t/qml-pyqt6-cant-get-native-control-appearance/1240/2?u=rokejulianlockhart) does? > > </strike> <img width="643" alt="Screenshot 2023-05-11 at 3 15 36 PM" src="https://github.com/discourse/discourse/assets/1555215/e7087502-b0a1-4219-be0b-ad3904cc6a6f"> Related: - ~~https://github.com/discourse/discourse/pull/21500~~ - https://github.com/discourse/discourse-footnote/pull/62 - https://github.com/discourse/discourse-bbcode/pull/50

  1) PrettyText provides safety for img bbcode
     Failure/Error: expect(cooked).to eq(html)

       expected: "<p><img src=\"http://aaa.com&lt;script&gt;alert(1);&lt;/script&gt;\" alt=\"\" role=\"presentation\"></p>"
            got: "<p><img src=\"http://aaa.com<script>alert(1);</script>\" alt=\"\" role=\"presentation\"></p>"

       (compared using ==)
     # ./spec/lib/pretty_text_spec.rb:2150:in `block (2 levels) in <main>'
     # ./spec/rails_helper.rb:358:in `block (2 levels) in <top (required)>'
     # /home/sam/.gem/ruby/3.2.1/gems/webmock-3.18.1/lib/webmock/rspec.rb:37:in `block (2 levels) in <top (required)>'

これは非常に危険な気がします…提起します…

nat · 2023 年 6 月 20 日午後 12:43

デバッグありがとうございます！以下の修正で対応しました。

github.com/discourse/discourse

DEV: use HTML5 version of loofah

main ← loofah

opened 02:35AM - 12 May 23 UTC

SamSaffron

+123 -115

https://meta.discourse.org/t/markdown-preview-and-result-differ/263878 The re…sult of this markdown had different results in the composer preview and the post. This is solved by updating Loofah to the latest version and using html5 fragments like our user had reported. While the change was only needed in [cooked_post_processor.rb](https://github.com/discourse/discourse/pull/21500/files#diff-67de7f44aa04f02ceba9770e5d83b4465add3bd4297be871f94a2233cd4831a7), I've updated the other areas of our codebase to also use the html5 fragment. > <strike> > > Additionally, how come > > ```qml > import QtQuick > import QtQuick.Controls 2.15 as QQC2 > import QtQuick.Layouts > import QtPositioning > ``` > > and prepension of `QQC2` before `Action`, `Button`, `ApplicationWindow`, and `Frame` doesn't work, whereas > > ```qml > import QtQuick > import QtQuick.Controls > import QtQuick.Layouts > import QtPositioning > import QtQuick.Controls 2.15 as QQC2 > ``` > > and addition of `QQC2.Button` (per https://discuss.kde.org/t/qml-pyqt6-cant-get-native-control-appearance/1240/2?u=rokejulianlockhart) does? > > </strike> <img width="643" alt="Screenshot 2023-05-11 at 3 15 36 PM" src="https://github.com/discourse/discourse/assets/1555215/e7087502-b0a1-4219-be0b-ad3904cc6a6f"> Related: - ~~https://github.com/discourse/discourse/pull/21500~~ - https://github.com/discourse/discourse-footnote/pull/62 - https://github.com/discourse/discourse-bbcode/pull/50

こちらで確認できます。

~~

追加で、なぜ
import QtQuick
import QtQuick.Controls 2.15 as QQC2
import QtQuick.Layouts
import QtPositioning
で Action、Button、ApplicationWindow、Frame の前に QQC2 を付けると機能せず、
import QtQuick
import QtQuick.Controls
import QtQuick.Layouts
import QtPositioning
import QtQuick.Controls 2.15 as QQC2
で QQC2.Button を追加すると機能するのか（[qml+pyqt6] Can't get native control appearance - #2 by carl - Help - KDE Discuss 参照）？

~~

nat · 2023 年 6 月 22 日午後 12:43

このトピックは2日後に自動的に閉じられました。返信はもう受け付けられません。

トピック		返信	表示
HTML headings with line breaks -> no heading and inconsistency between preview and rendered post Bug	9	1285	2023 年 6 月 22 日
"Traditional markdown linebreaks" not reflected in preview Bug	4	1342	2017 年 7 月 3 日
Strangely ordered markdown + HTML is shown differently in post vs. preview Support	7	628	2023 年 2 月 2 日
Multi-Line Strikethrough Not Covering Quotes Support	24	4570	2020 年 9 月 4 日
Quoting not working Support	74	3981	2021 年 4 月 6 日

Markdownのプレビューと結果が異なります

try.discourse.org

discuss.kde.org

関連トピック