[openstreetmap/openstreetmap-website] Integrate human-readable wikidata browser plugin (PR #6127)
Marwin Hochfelsner
notifications at github.com
Thu Jul 3 20:56:56 UTC 2025
@hlfan commented on this pull request.
> + .append($("<img>").attr({ src: OSM.WIKIMEDIA_COMMONS_URL + "Special:FilePath/" + data.icon, height: "32" }))
+ .addClass("float-end mb-1 ms-2")
+ .appendTo(cell);
+ }
+ if (data.label) {
+ $btn
+ .siblings(`a[href*="wikidata.org/entity/${data.qid}"]`)
+ .clone()
+ .text(data.label)
+ .addClass("me-1")
+ .appendTo(cell);
+ }
+ if (data.article) {
+ $(`<${data.label ? "sup" : "div"}>`)
+ .append($("<a>")
+ .attr("href", `https://${data.article.site.slice(0, -4)}.wikipedia.org/wiki/` + encodeURIComponent(data.article.title))
How about linking the label to the article to
1. not have redundant links and
2. not have to think about how to tell the user about the article?
Also if we use wikipedia.svg it should be inlined to be able to use currentColor.
--
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/6127#discussion_r2183740748
You are receiving this because you are subscribed to this thread.
Message ID: <openstreetmap/openstreetmap-website/pull/6127/review/2984655132 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20250703/5c4fca6f/attachment-0001.htm>
More information about the rails-dev
mailing list