bh 6.9.2 → 6.10.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: a00566adde28789768cc5886928cea8beda894754e6e69a02f92b9d2f1b04eda
-  data.tar.gz: e145e5153def494578b3ece720f11cb3e1ce89a772b09507b75b3dc9d79772af
+  metadata.gz: 1698ddd9bc4657e18402fafa6a96ade1e5cebc16e81bb38f3dcdef8bbbf268f8
+  data.tar.gz: 27ae0e2709e2c8218489233b5501d3098cdfa8dcda5e60ac770aeb2354ba8bab
 SHA512:
-  metadata.gz: eb9e51b70d0c16d6a51337ebe4f73333aee4d5ebda852308438718e7059ea3d4818d2510f7741d09f261ee708d44eb71ba9c8ed0f43fa65d461305507334b5c2
-  data.tar.gz: 1d816cbefc43df66146259442266a68c21706479c98936163ddb51034d0f60ecca24f1b294619f013f8f397ae0fdd6ed6ffdde0d51f8ba711ad0f755c0d9572b
+  metadata.gz: c0cac35a163c597c2f14e84f79e6cefe1b0eb6ad2e08deba97162a1d0dad7dd3fc0d0a58a72180028d51f42a8e7c607d912cb0df8482c7fa1ed2805fe82f3585
+  data.tar.gz: 200643a93cea4bbfc54ca78679362042e5565525667573478624af65aaf68935565fb774981c947ed4ea1c1745e996ac1154c69bb9f5e845065c4ccb26e137e2

data/CHANGELOG.md

@@ -8,6 +8,24 @@ For more information about changelogs, check
 
 ## [Unreleased]
 
+## 6.10.1 - 2026-09-26
+
+* [FIX] A tooltip closes when its icon, or the button or link holding it, is clicked
+
+  A click that takes the icon away — the sidebar's moon and sun swap places — left the tooltip
+  open with nothing under the pointer to close it, and Popper put it in the page's corner,
+  where a hidden element is. It is closed in the capture phase, before the click's own action
+  changes the page.
+
+## 6.10.0 - 2026-09-26
+
+* [FEATURE] A pin or an area on a map leads where its row does
+
+  The map controller takes a point as `[lat, lng, href, title]` and a place as
+  `[id, href, title]`. A pin with an `href` is clickable, shows its `title` on hover, and a
+  click follows it through Turbo; an area filled on a boundary layer is followed the same
+  way. A bare `[lat, lng]` or a bare place ID still draws a pin that leads nowhere.
+
 ## 6.9.2 - 2026-09-20
 
 * [FIX] A small select's combobox is small too

data/app/javascript/bh/map_controller.js

@@ -6,6 +6,11 @@ import { load } from './map/loader.js'
 // none, a pin at the place; a row keeping coordinates is a pin there and then, with
 // nothing to look up. The key and the map are the host's, read from its credentials; the
 // rows are the page's, and the map is fitted round whatever it drew.
+//
+// A row may carry where it leads and what it is called: a point as `[lat, lng, href, title]`,
+// a place as `[id, href, title]`, where a bare place ID or a bare `[lat, lng]` leads nowhere.
+// A pin or an area with somewhere to lead is clicked through to it, the way the row's own
+// link in the table is.
 export default class extends Controller {
   static values = { key: String, id: String, boundary: String, places: Array, points: Array }
 
@@ -30,7 +35,8 @@ export default class extends Controller {
   async place(map, bounds) {
     if (this.placesValue.length === 0) return
     const { Place } = await google.maps.importLibrary('places')
-    const places = this.placesValue.map(id => new Place({ id }))
+    const rows = this.placesValue.map(row => [].concat(row))
+    const places = rows.map(([id, href, title]) => ({ place: new Place({ id }), href, title }))
 
     if (this.hasBoundaryValue) return this.fill(map, bounds, places)
     return this.pin(map, bounds, places)
@@ -38,13 +44,19 @@ export default class extends Controller {
 
   // The layer styles every boundary Google knows at that level, and a function saying
   // which of them are ours is what fills them in. The bounds are the places' viewports.
+  // An area with somewhere to lead is followed there when clicked.
   async fill(map, bounds, places) {
-    const ours = new Set(this.placesValue)
-    map.getFeatureLayer(this.boundaryValue).style = ({ feature }) => {
-      if (ours.has(feature.placeId)) return FILLED
+    const hrefs = new Map(places.map(({ place, href }) => [place.id, href]))
+    const layer = map.getFeatureLayer(this.boundaryValue)
+    layer.style = ({ feature }) => {
+      if (hrefs.has(feature.placeId)) return FILLED
     }
+    layer.addListener('click', ({ features }) => {
+      const href = features.map(feature => hrefs.get(feature.placeId)).find(Boolean)
+      if (href) visit(href)
+    })
 
-    await Promise.all(places.map(place =>
+    await Promise.all(places.map(({ place }) =>
       fetched(place, 'viewport').then(() => { if (place.viewport) bounds.union(place.viewport) })
     ))
   }
@@ -52,8 +64,8 @@ export default class extends Controller {
   async pin(map, bounds, places) {
     const drop = await this.dropper(map, bounds)
 
-    await Promise.all(places.map(place =>
-      fetched(place, 'location').then(() => { if (place.location) drop(place.location) })
+    await Promise.all(places.map(({ place, href, title }) =>
+      fetched(place, 'location').then(() => { if (place.location) drop(place.location, href, title) })
     ))
   }
 
@@ -62,20 +74,29 @@ export default class extends Controller {
     if (this.pointsValue.length === 0) return
     const drop = await this.dropper(map, bounds)
 
-    for (const [lat, lng] of this.pointsValue) drop({ lat, lng })
+    for (const [lat, lng, href, title] of this.pointsValue) drop({ lat, lng }, href, title)
   }
 
-  // One marker at a position, and the bounds widened to hold it.
+  // One marker at a position, and the bounds widened to hold it. A marker with somewhere to
+  // lead is clickable, names what it leads to on hover, and is followed there when clicked.
   async dropper(map, bounds) {
     const { AdvancedMarkerElement } = await google.maps.importLibrary('marker')
 
-    return (position) => {
-      new AdvancedMarkerElement({ map, position })
+    return (position, href, title) => {
+      const marker = new AdvancedMarkerElement({ map, position, title, gmpClickable: Boolean(href) })
+      if (href) marker.addEventListener('gmp-click', () => visit(href))
       bounds.extend(position)
     }
   }
 }
 
+// Through Turbo where the page has it, so the table's frame and scroll behave as a click on
+// the row's own link would; a plain navigation otherwise.
+function visit(href) {
+  if (window.Turbo) window.Turbo.visit(href)
+  else window.location.assign(href)
+}
+
 // A place that cannot be fetched — an ID Google no longer knows — is left off the map
 // rather than taking the rest of the page's rows with it.
 function fetched(place, field) {

data/app/javascript/bh/tooltip_controller.js

@@ -10,14 +10,24 @@ import { Tooltip } from 'bootstrap'
 // is also what shows a tooltip, and a word popping up over a tap reads as a hitch.
 // Asked two ways, since an iPhone answered the first with a tooltip all the same: a
 // screen that cannot hover, or a pointer too coarse to rest on one word of a row.
+//
+// Closed by a click on the icon or on whatever holds it, a button or a link: what a click
+// does may take the icon away — the sidebar's moon and sun swap places, a sorted heading
+// is drawn again — and a tooltip whose icon is gone from under the pointer sees no mouse
+// leave, stays open, and is put by Popper wherever a hidden element is: the page's corner.
+// In the capture phase, so it closes before the click's own action changes the page.
 export default class extends Controller {
   connect() {
     if (matchMedia('(hover: none), (pointer: coarse)').matches) return
 
     this.tooltip = Tooltip.getOrCreateInstance(this.element)
+    this.holder = this.element.closest('button, a') || this.element
+    this.close = () => this.tooltip.hide()
+    this.holder.addEventListener('click', this.close, true)
   }
 
   disconnect() {
+    this.holder?.removeEventListener('click', this.close, true)
     this.tooltip?.dispose()
   }
 }

data/lib/bh/version.rb

@@ -1,4 +1,4 @@
 module Bh
   # Version of the gem, read by the gemspec and by hosts checking compatibility.
-  VERSION = '6.9.2'
+  VERSION = '6.10.1'
 end