andrewharvey left a comment (openstreetmap/openstreetmap-website#5780)

> There is a difference with changing how things that are already detected as 
> links look and the rest. You don't expect "I added a note 5 years ago" to 
> link to note https://github.com/openstreetmap/openstreetmap-website/pull/5 
> but that's what you'll get with those formats.

Requiring a minimum number of digits would cover most false positives like 
that, we don't need it to work on note number 5.

> GitHub gets around this problem by requiring a # in front of the number, or 
> the full URL. Even just detecting and linkifying the URL would be a 
> significant improvement.

Agreed, we could start by only doing note/X, noteX and the full note URL (where 
X is > 4 characters).

Personally I'm not a fan of the shorterned osm.org domain, as pointed out on 
#5844 people might be be confused if osm.org is the official openstreetmap.org 
or if it's a different site.

I still think we should go further per my proposed here and remove the domain 
name entirely in the display text, while still allowing the link to work when 
text is copy pasted.

-- 
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/issues/5780#issuecomment-2763048895
You are receiving this because you are subscribed to this thread.

Message ID: 
<openstreetmap/openstreetmap-website/issues/5780/2763048...@github.com>
_______________________________________________
rails-dev mailing list
rails-dev@openstreetmap.org
https://lists.openstreetmap.org/listinfo/rails-dev

Reply via email to