@hlfan commented on this pull request.
> @@ -87,18 +87,33 @@ def sanitize(text)
Sanitize.clean(text, Sanitize::Config::OSM).html_safe
end
- def linkify(text, mode = :urls)
- ERB::Util.html_escape(text)
- .then { |html| expand_link_shorthands(html) }
- .then { |html| expand_host_shorthands(html) }
- .then { |html| auto_link(html, mode) }
- .html_safe
+ def linkify(text, mode = :urls, hosts: true, paths: true)
+ link_attr = 'rel="nofollow noopener noreferrer" dir="auto"'
+ html = ERB::Util.html_escape(text)
+
+ html = expand_link_shorthands(html) if paths
+ html = expand_host_shorthands(html) if hosts
Arguably, html shouldn't have osm link hosts shortened, but I don't know if
html richtext is even used here.
--
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/6720#discussion_r2709561235
You are receiving this because you are subscribed to this thread.
Message ID:
<openstreetmap/openstreetmap-website/pull/6720/review/[email protected]>_______________________________________________
rails-dev mailing list
[email protected]
https://lists.openstreetmap.org/listinfo/rails-dev
