[openstreetmap/openstreetmap-website] Add dir="auto" to Markdown content for bidi (PR #5840)

Nitai Sasson notifications at github.com
Sun Mar 23 20:38:20 UTC 2025


@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/2708763958 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20250323/2bb195b2/attachment.htm>


More information about the rails-dev mailing list