@tomhughes commented on this pull request.
> @@ -112,7 +116,10 @@ OSM.History = function (map) {
function toggleChangesetHighlight(id, state) {
changesetsLayer.toggleChangesetHighlight(id, state);
- $("#changeset_" + id).toggleClass("selected", state);
+ $("#sidebar_content .changesets ol li").removeClass("selected");
+ if (state) {
+ $("#changeset_" + id).addClass("selected");
+ }
Do we need a similar change inside the `toggleChangesetHighlight` call on the
layer or is there something that stops that getting "stuck" in the same way?
--
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/6059#pullrequestreview-2871993509
You are receiving this because you are subscribed to this thread.
Message ID:
<openstreetmap/openstreetmap-website/pull/6059/review/2871993...@github.com>_______________________________________________
rails-dev mailing list
rails-dev@openstreetmap.org
https://lists.openstreetmap.org/listinfo/rails-dev
