@NeatNit commented on this pull request.


> @@ -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)

Oops, wrong link. This is the one: 
https://kramdown.gettalong.org/rdoc/Kramdown/Converter/Base.html

> Implementing a new converter is rather easy: just derive a new class from 
> this class and put it in the 
> [Kramdown::Converter](https://kramdown.gettalong.org/rdoc/Kramdown/Converter.html)
>  module (the latter is only needed if auto-detection should work properly).

Basically the same thing.

kramdown converts "html_bidi" to "HtmlBidi" and picks the class using that.

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

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

Reply via email to