@hlfan commented on this pull request.
> @@ -101,4 +112,108 @@
scrollableList.scrollLeft = scrollableList.scrollWidth -
scrollableList.offsetWidth;
}
}
+
+ function previewWikidataValue($btn) {
+ if (!OSM.WIKIDATA_API_URL) return;
+ const items = $btn.data("qids");
+ if (!items?.length) return;
+ $btn.addClass("disabled").removeClass("wdt-preview");
I'd like the button when focused to have an outline as that's more visible than
the slight blue background.
The state management here is to not trigger the listener when disabled, but
changing the selector to `...:not(:disabled)` is probably better.
--
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/6127#discussion_r2208624940
You are receiving this because you are subscribed to this thread.
Message ID:
<openstreetmap/openstreetmap-website/pull/6127/review/3022240...@github.com>_______________________________________________
rails-dev mailing list
rails-dev@openstreetmap.org
https://lists.openstreetmap.org/listinfo/rails-dev
