bh 6.9.1 → 6.10.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 29a85e5fc78e270eb30a28d5b6ac8e815b7f8ab8dc14a961080b4e11f5b7a6ce
-  data.tar.gz: 92fe0b26cf3e684106552ce907b92a47b5b030cc33e5ddaf83d4f81e139b9704
+  metadata.gz: 323e18fad01eb3fdc146019166fdec96ecc7455a12b2ce170525ee07d5083c47
+  data.tar.gz: c6050641e6910669ef9b645b55cae1141f94370e9a3167ddd93937c4f28274eb
 SHA512:
-  metadata.gz: 8c7f04aba5850ee986b0eec80c713049629b17eff4bdd922e7e981eaa0aeedb1f0d2776d0b263b2db61a26a1f6336667c4eb0b8b11b8ef633e5acaab19cd9298
-  data.tar.gz: a65ea0eff219a4d095f6ee0b4215656c34ab17cde0b8ce0ea0a883f590014bd6d47c33756c99c83839cf28d2f84b065a2dd7d7bd459577bd5e429574f51160f1
+  metadata.gz: 901c770b2d53376cd82a3ccf7f0e09c3499f26415d3f10dc896488b483b344cfd6cef5d3eed869174337945338c3b79fc6c38e36f5ce02273d5ba6caad06a35a
+  data.tar.gz: 15f84dd6790385a89d6d8ca6b9fcceab7168ce0c8dcae1a63cc24e749c282a782d18f118f86a0e1fc85ee4ca94c2ec8d4d4634110abba83b839eca32f7da14bc

data/CHANGELOG.md

@@ -8,6 +8,24 @@ For more information about changelogs, check
 
 ## [Unreleased]
 
+## 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
+
+  The button that stands in for a `<select>` took the compact size from `form-select-sm`
+  alone, so a select wearing `form-control-sm` — which is what a filter beside a search
+  box wears — was replaced by a full-height one. It stood two pixels taller than the trail
+  beside it, which is what set the height of every row it sat in.
+
 ## 6.9.1 - 2026-09-20
 
 * [FIX] A combobox's search box carries an id

data/app/javascript/bh/combobox/menu.js

@@ -5,7 +5,7 @@
 // The toggle takes the select's size and its error, and is labelled the way the select was.
 export function menuFor(select, controller) {
   const toggle = element('button', 'form-control combobox-toggle', { type: 'button' })
-  if (select.classList.contains('form-select-sm')) { toggle.classList.add('form-control-sm') }
+  if (['form-select-sm', 'form-control-sm'].some(size => select.classList.contains(size))) { toggle.classList.add('form-control-sm') }
   if (select.classList.contains('is-invalid')) { toggle.classList.add('is-invalid') }
   toggle.dataset.bsToggle = 'combobox'
   toggle.dataset.bsSearch = 'true'

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/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.1'
+  VERSION = '6.10.0'
 end