- Issue created by @waako
- Status changed to Needs review
10 months ago 3:36pm 8 March 2024 - First commit to issue fork.
- Status changed to Fixed
10 months ago 2:51pm 10 March 2024 - 🇮🇹Italy itamair
Thanks for reporting this @waako ...
This doesn't look a big deal to me (but rather a kind of picky one), also because this doesn't happen on a general basis,
but ok ... it makes some sense to me, and I agree to merge your MR.
This will be part of the next Leaflet module release. - Status changed to Needs review
10 months ago 12:42pm 11 March 2024 - 🇫🇷France waako Normandie
Thanks @itamair
Really appreciate the quick approval and merge!I do understand that it may not seem like a big deal to most people, but for users with disabilities such as cognitive or motor impairments, this can be very disorientating and make it challenging to navigate back.
Plus as you say, "it doesn't happen on a general basis", so may well keep it consistent with not opening links in new windows unexpectedly, none of us like our clicks to be hikacked :)For background context, leaflet is the default mapping provider for https://localgovdrupal.org/ and this was causing accessibility errors for many local authority websites, so they were keen to get this resolved.
Thanks again.
- Status changed to Fixed
10 months ago 12:42pm 11 March 2024 - 🇩🇰Denmark ressa Copenhagen
Thanks @waako and @itamair, I agree with removing
target="_blank"
and have just updated 📌 Stamen map tiles moving to Stadia hosting and will no longer be available at old URL Needs review to follow this practice as well in that MR. Automatically closed - issue fixed for 2 weeks with no activity.