@tomhughes commented on this pull request.


> @@ -1,5 +1,45 @@
 # frozen_string_literal: true
 
+module Kramdown

Injecting code into the `Kramdown` namespace is not ideal as we don't own that.

> @@ -96,7 +136,7 @@ def to_text
 
   class Markdown < Base
     def to_html
-      linkify(sanitize(document.to_html), :all)
+      linkify(sanitize(document.to_html_bidi), :all)

How does this work? I don't see anything defining a `to_html_bidi` method?

-- 
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/5840#pullrequestreview-2708751422
You are receiving this because you are subscribed to this thread.

Message ID: 
<openstreetmap/openstreetmap-website/pull/5840/review/2708751...@github.com>
_______________________________________________
rails-dev mailing list
rails-dev@openstreetmap.org
https://lists.openstreetmap.org/listinfo/rails-dev

Reply via email to