@lblod/ember-rdfa-editor-lblod-plugins 28.2.1 → 29.0.0

package/CHANGELOG.md

@@ -1,5 +1,25 @@
 # @lblod/ember-rdfa-editor-lblod-plugins
 
+## 29.0.0
+
+### Major Changes
+
+- [#554](https://github.com/lblod/ember-rdfa-editor-lblod-plugins/pull/554) [`91e5c25`](https://github.com/lblod/ember-rdfa-editor-lblod-plugins/commit/91e5c2540fb66aa4a8356a0e3977ff7ab7655440) Thanks [@elpoelma](https://github.com/elpoelma)! - Remove obsolete `webpack-config.js`. This config is solely needed for the tests, and is now inlined in the `ember-cli-build` file of the test-app
+
+### Patch Changes
+
+- [#556](https://github.com/lblod/ember-rdfa-editor-lblod-plugins/pull/556) [`aa0c182`](https://github.com/lblod/ember-rdfa-editor-lblod-plugins/commit/aa0c1824811712fa6052f5962a2b8473d0bab7a4) Thanks [@piemonkey](https://github.com/piemonkey)! - Bundle location plugin marker icons to avoid issues when bundling the code for embeddable
+
+- [`867ca46`](https://github.com/lblod/ember-rdfa-editor-lblod-plugins/commit/867ca46b680b2fdc8c5c41481a839f43968ebfd5) Thanks [@abeforgit](https://github.com/abeforgit)! - Always render a period after structure names, it should never be a colon
+
+- [#554](https://github.com/lblod/ember-rdfa-editor-lblod-plugins/pull/554) [`bc1a294`](https://github.com/lblod/ember-rdfa-editor-lblod-plugins/commit/bc1a29474c4fd09367bfc08c8e97439cbba27198) Thanks [@elpoelma](https://github.com/elpoelma)! - Update `ember-template-imports` to version 4.3.0 . This version unlocks `.gts`/`.gjs` sourcemaps.
+
+## 28.2.2
+
+### Patch Changes
+
+- [`261f2a3`](https://github.com/lblod/ember-rdfa-editor-lblod-plugins/commit/261f2a39b5b67c6ba373e3b35a91c242d27a4277) Thanks [@piemonkey](https://github.com/piemonkey)! - Bundle location plugin marker icons to avoid issues when bundling the code for embeddable
+
 ## 28.2.1
 
 ### Patch Changes

package/addon/components/location-plugin/map.gts

@@ -6,6 +6,7 @@ import and from 'ember-truth-helpers/helpers/and';
 import eq from 'ember-truth-helpers/helpers/eq';
 import t from 'ember-intl/helpers/t';
 import {
+  Leaflet,
   LeafletMap,
   type LeafletMapSig,
   type LeafletMapStart,
@@ -20,6 +21,38 @@ import {
   type GlobalCoordinates,
 } from '@lblod/ember-rdfa-editor-lblod-plugins/plugins/location-plugin/utils/geo-helpers';
 
+// Taken from Leaflet's default icon, with the viewbox tweaked (from 0 0 500 820), so that the icon
+// is a whole number of pixels in size, to make positioning the point easy.
+const ICON_HTML = `<svg viewBox="0 0 500 800" version="1.1" xmlns="http://www.w3.org/2000/svg" xml:space="preserve"
+     style="fill-rule: evenodd; clip-rule: evenodd; stroke-linecap: round;">
+    <defs>
+        <linearGradient x1="0" y1="0" x2="1" y2="0" gradientUnits="userSpaceOnUse" gradientTransform="matrix(2.30025e-15,-37.566,37.566,2.30025e-15,416.455,540.999)" id="map-marker-38-f">
+            <stop offset="0" stop-color="rgb(18,111,198)"/>
+            <stop offset="1" stop-color="rgb(76,156,209)"/>
+        </linearGradient>
+        <linearGradient x1="0" y1="0" x2="1" y2="0"
+                        gradientUnits="userSpaceOnUse"
+                        gradientTransform="matrix(1.16666e-15,-19.053,19.053,1.16666e-15,414.482,522.486)"
+                        id="map-marker-38-s">
+            <stop offset="0" stop-color="rgb(46,108,151)"/>
+            <stop offset="1" stop-color="rgb(56,131,183)"/>
+        </linearGradient>
+    </defs>
+    <g transform="matrix(19.5417,0,0,19.5417,-7889.1,-9807.44)">
+        <path fill="#FFFFFF" d="M421.2,515.5c0,2.6-2.1,4.7-4.7,4.7c-2.6,0-4.7-2.1-4.7-4.7c0-2.6,2.1-4.7,4.7-4.7 C419.1,510.8,421.2,512.9,421.2,515.5z"/>
+        <path d="M416.544,503.612C409.971,503.612 404.5,509.303 404.5,515.478C404.5,518.256 406.064,521.786 407.194,524.224L416.5,542.096L425.762,524.224C426.892,521.786 428.5,518.433 428.5,515.478C428.5,509.303 423.117,503.612 416.544,503.612ZM416.544,510.767C419.128,510.784 421.223,512.889 421.223,515.477C421.223,518.065 419.128,520.14 416.544,520.156C413.96,520.139 411.865,518.066 411.865,515.477C411.865,512.889 413.96,510.784 416.544,510.767Z" stroke-width="1.1px" fill="url(#map-marker-38-f)" stroke="url(#map-marker-38-s)"/>
+    </g>
+</svg>`;
+// We use Leaflet.divIcon directly, as passing arguments to the ember-leaflet helper seems to be
+// broken
+const MARKER_ICON = Leaflet.divIcon({
+  html: ICON_HTML,
+  // This prevents the default class from being applied, along with the default styling
+  className: '',
+  iconAnchor: [15, 48],
+  iconSize: [30, 48],
+});
+
 export const SUPPORTED_LOCATION_TYPES = ['address', 'place', 'area'] as const;
 export type LocationType = (typeof SUPPORTED_LOCATION_TYPES)[number];
 
@@ -248,7 +281,11 @@ export default class LocationPluginMapComponent extends Component<Signature> {
   }
 
   <template>
-    <div class='map-wrapper' ...attributes>
+    <div
+      class='map-wrapper
+        {{unless (eq @locationType "address") "map-cursor-pointer"}}'
+      ...attributes
+    >
       <MapWrapper
         @mapStart={{this.mapLocation}}
         @onClick={{this.onMapClick}}
@@ -259,7 +296,11 @@ export default class LocationPluginMapComponent extends Component<Signature> {
           @attribution={{MAP_TILE_ATTRIBUTION}}
         />
         {{#if (and @address (eq @locationType 'address') this.foundAddress)}}
-          <layers.marker @location={{this.foundAddress}} as |marker|>
+          <layers.marker
+            @location={{this.foundAddress}}
+            @icon={{MARKER_ICON}}
+            as |marker|
+          >
             <marker.tooltip>
               {{@address.formatted}}
             </marker.tooltip>
@@ -269,6 +310,7 @@ export default class LocationPluginMapComponent extends Component<Signature> {
           <layers.marker
             @opacity={{if @location 0.3 1}}
             @location={{this.existingLocationCoords}}
+            @icon={{MARKER_ICON}}
             as |marker|
           >
             <marker.tooltip>
@@ -277,7 +319,7 @@ export default class LocationPluginMapComponent extends Component<Signature> {
           </layers.marker>
         {{/if}}
         {{#if (and @location (eq @locationType 'place'))}}
-          <layers.marker @location={{@location.global}} />
+          <layers.marker @location={{@location.global}} @icon={{MARKER_ICON}} />
         {{/if}}
         {{#if (eq @locationType 'area')}}
           <layers.polyline @locations={{this.vertices}} />
@@ -285,6 +327,7 @@ export default class LocationPluginMapComponent extends Component<Signature> {
             <layers.marker
               @location={{vertex}}
               @onClick={{fn this.onVertexClick index}}
+              @icon={{MARKER_ICON}}
               as |marker|
             >
               {{#if (eq index 0)}}

package/addon/plugins/lpdc-plugin/index.ts

@@ -1,2 +1,2 @@
-export { type LpdcPluginConfig, LPDC } from './types';
+export type { LpdcPluginConfig, LPDC } from './types';
 export { fetchLpdcs } from './api';

package/app/styles/location-plugin.scss

@@ -21,6 +21,11 @@
     height: 100%;
   }
 
+  // Override the 'grab' cursor to 'pointer' so we can accurately pick points
+  .map-cursor-pointer > .leaflet-grab {
+    cursor: pointer;
+  }
+
   .leaflet-container {
     height: 100%;
   }

package/declarations/addon/plugins/lpdc-plugin/index.d.ts

@@ -1,2 +1,2 @@
-export { type LpdcPluginConfig, LPDC } from './types';
+export type { LpdcPluginConfig, LPDC } from './types';
 export { fetchLpdcs } from './api';

package/index.js

@@ -13,9 +13,6 @@ module.exports = {
       sourceMaps: 'inline',
       plugins: [],
     },
-    autoImport: {
-      webpack: require('./webpack-config'),
-    },
   },
 
   init() {
@@ -33,7 +30,6 @@ module.exports = {
       // ember-concurrency v4+ requires a custom babel transform. Once we drop ember-concurrency v3 support we can remove this conditional registration.
       this.options.babel.plugins.push(
         // require is missing as this lib uses a too old ember-concurrency version
-        // eslint-disable-next-line node/no-missing-require
         require.resolve('ember-concurrency/async-arrow-task-transform'),
       );
     }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@lblod/ember-rdfa-editor-lblod-plugins",
-  "version": "28.2.1",
+  "version": "29.0.0",
   "description": "Ember addon providing lblod specific plugins for the ember-rdfa-editor",
   "keywords": [
     "ember-addon",
@@ -74,7 +74,7 @@
     "ember-cli-version-checker": "^5.1.2",
     "ember-mu-transform-helpers": "^2.1.2",
     "ember-resources": "^7.0.2",
-    "ember-template-imports": "^4.1.1",
+    "ember-template-imports": "^4.3.0",
     "ember-velcro": "^2.2.0",
     "fetch-sparql-endpoint": "^3.3.3",
     "n2words": "^1.21.0",
@@ -97,7 +97,11 @@
     "@ember/render-modifiers": "^2.1.0",
     "@ember/string": "^3.1.1",
     "@ember/test-helpers": "^3.3.1",
+    "@embroider/broccoli-side-watch": "^1.0.1",
+    "@embroider/compat": "^3.8.4",
+    "@embroider/core": "^3.5.4",
     "@embroider/test-setup": "^4.0.0",
+    "@embroider/webpack": "^4.1.0",
     "@gavant/glint-template-types": "^0.4.0",
     "@glimmer/component": "^1.1.2",
     "@glimmer/tracking": "^1.1.2",
@@ -135,6 +139,7 @@
     "ember-cli-sass": "^11.0.1",
     "ember-cli-sri": "^2.1.1",
     "ember-cli-terser": "^4.0.2",
+    "ember-concurrency": "^4.0.2",
     "ember-disable-prototype-extensions": "^1.1.3",
     "ember-element-helper": "^0.8.6",
     "ember-eslint-parser": "^0.5.7",
@@ -142,7 +147,6 @@
     "ember-leaflet": "^5.1.3",
     "ember-load-initializers": "^2.1.2",
     "ember-modifier": "~4.1.0",
-    "ember-concurrency": "^4.0.2",
     "ember-page-title": "^8.2.3",
     "ember-power-select": "~7.1.0",
     "ember-qunit": "^8.1.0",
@@ -171,6 +175,7 @@
     "sass": "^1.77.6",
     "sinon": "^17.0.1",
     "typescript": "^5.6.2",
+    "vm-browserify": "^1.1.2",
     "webpack": "^5.95.0"
   },
   "peerDependencies": {