Merge pull request #228 from jonataswalker/update-demo

fix: add IIFE version & update fiddle
jonataswalker · Sep 22, 2022 · a146954 · a146954
2 parents ed222ec + d1281b1
commit a146954
Show file tree

Hide file tree

Showing 6 changed files with 35 additions and 16 deletions.
diff --git a/README.md b/README.md
@@ -21,7 +21,7 @@ A `contextmenu` extension for [OpenLayers](http://openlayers.org/). **Requires**
 
 ## Demo
 
-You can see [here a demo](http://rawgit.com/jonataswalker/ol-contextmenu/master/examples/contextmenu.html) or [JSFiddle](https://jsfiddle.net/jonataswalker/ooxs1w5d/).
+[JSFiddle](https://jsfiddle.net/jonataswalker/ooxs1w5d/).
 
 ## How to use it?
 

diff --git a/examples/contextmenu.html b/examples/contextmenu.html
@@ -10,7 +10,7 @@
     <body>
         <div id="map" tabindex="0"></div>
         <script src="https://cdn.jsdelivr.net/npm/[email protected]/dist/ol.js"></script>
-        <script src="../dist/ol-contextmenu.umd.cjs"></script>
+        <script src="../dist/ol-contextmenu.iife.js"></script>
         <script src="contextmenu.js"></script>
     </body>
 </html>
diff --git a/package-lock.json b/package-lock.json
diff --git a/package.json b/package.json
@@ -1,9 +1,10 @@
 {
     "name": "ol-contextmenu",
-    "version": "5.0.0",
+    "version": "5.0.3",
     "description": "Custom Context Menu for Openlayers",
     "type": "module",
     "main": "./dist/ol-contextmenu.umd.cjs",
+    "browser": "./dist/ol-contextmenu.iife.js",
     "module": "./dist/ol-contextmenu.js",
     "exports": {
         ".": {
@@ -60,6 +61,7 @@
         "rimraf": "^3.0.2",
         "typescript": "^4.8.3",
         "vite": "^3.1.3",
+        "vite-plugin-banner": "^0.5.0",
         "vite-plugin-dts": "^1.5.0"
     },
     "peerDependencies": {

diff --git a/src/main.ts b/src/main.ts
@@ -339,12 +339,3 @@ export default class ContextMenu extends Control {
         }
     }
 }
-
-// Expose ContextMenu as ol.control.ContextMenu if using a full build of OpenLayers
-// eslint-disable-next-line @typescript-eslint/ban-ts-comment
-// @ts-ignore
-if (window.ol?.control) {
-    // eslint-disable-next-line @typescript-eslint/ban-ts-comment
-    // @ts-ignore
-    window.ol.control.ContextMenu = ContextMenu;
-}
diff --git a/vite.config.ts b/vite.config.ts
@@ -2,6 +2,7 @@ import { readFileSync } from 'fs';
 
 import { defineConfig } from 'vite';
 import dts from 'vite-plugin-dts';
+import bannerPlugin from 'vite-plugin-banner';
 
 import { CSS_CLASSES } from './src/constants';
 
@@ -14,6 +15,14 @@ const globals = {
     ol: 'ol',
 };
 
+const banner = `
+  /*!
+  * ${pkg.name} - v${pkg.version}
+  * ${pkg.homepage}
+  * Built: ${new Date()}
+  */
+`;
+
 const additionalData = Object.keys(CSS_CLASSES).reduce(
     (accumulator, current) => `${accumulator}$${current}:${CSS_CLASSES[String(current)]};`,
     ''
@@ -35,13 +44,17 @@ export default defineConfig(({ command }) =>
                       entry: './src/main.ts',
                       name: 'ContextMenu',
                       fileName: 'ol-contextmenu',
+                      formats: ['es', 'umd', 'iife'],
                   },
                   rollupOptions: {
                       external,
-                      output: { globals, assetFileNames: () => 'ol-contextmenu.css' },
+                      output: {
+                          globals,
+                          assetFileNames: () => 'ol-contextmenu.css',
+                      },
                   },
               },
-              plugins: [dts({ insertTypesEntry: true })],
+              plugins: [dts({ insertTypesEntry: true }), bannerPlugin(banner)],
               define: {
                   __APP_VERSION__: JSON.stringify(pkg.version),
               },
-Original file line number
+Diff line change
@@ Expand Up @@
     ## Demo
-    You can see [here a demo](http://rawgit.com/jonataswalker/ol-contextmenu/master/examples/contextmenu.html) or [JSFiddle](https://jsfiddle.net/jonataswalker/ooxs1w5d/).
+    [JSFiddle](https://jsfiddle.net/jonataswalker/ooxs1w5d/).
     ## How to use it?
@@ Expand Down @@