Skip to content

Commit

Permalink
Use importmap for the geo viewer
Browse files Browse the repository at this point in the history
  • Loading branch information
jcoyne committed Jan 30, 2025
1 parent a5925b8 commit 7117795
Show file tree
Hide file tree
Showing 10 changed files with 24 additions and 23 deletions.
1 change: 0 additions & 1 deletion app/components/embed/geo_component.html.erb
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,6 @@
<%= content_tag :div, viewer.map_element_options do %>

<% end %>
<%= javascript_pack_tag('geo') %>
</div>

<%= render 'embed/metadata_panel', viewer: viewer %>
Expand Down
10 changes: 5 additions & 5 deletions app/javascript/geo.js
Original file line number Diff line number Diff line change
@@ -1,10 +1,10 @@
import EmbedThis from "../src/modules/embed_this"
import PopupPanels from "../src/modules/popup_panels"
import GeoViewer from "../src/modules/geo_viewer"
import { trackView } from "../src/modules/metrics"
import EmbedThis from "src/modules/embed_this"
import PopupPanels from "src/modules/popup_panels"
import GeoViewer from "src/modules/geo_viewer"
import { trackView } from "src/modules/metrics"

import "leaflet"
import "../../../vendor/assets/javascripts/Leaflet.Control.Custom"
import "Leaflet.Control.Custom"

document.addEventListener("DOMContentLoaded", () => {
document.getElementById("sul-embed-object").hidden = false
Expand Down
15 changes: 0 additions & 15 deletions app/javascript/packs/geo.js

This file was deleted.

3 changes: 2 additions & 1 deletion app/javascript/src/modules/geo_viewer.js
Original file line number Diff line number Diff line change
@@ -1,4 +1,5 @@
import './leaflet_opacity'
import 'src/modules/leaflet_opacity'

export default {
isDefined: function (object) {
return typeof object !== 'undefined'
Expand Down
4 changes: 4 additions & 0 deletions app/viewers/embed/viewer/geo.rb
Original file line number Diff line number Diff line change
Expand Up @@ -7,6 +7,10 @@ def component
GeoComponent
end

def importmap
'geo'
end

##
# Options for the map element tag
# @return [Hash]
Expand Down
1 change: 1 addition & 0 deletions app/views/embed/iframe.html.erb
Original file line number Diff line number Diff line change
Expand Up @@ -17,6 +17,7 @@
});
</script>
<%= render 'analytics' %>
<!-- DOOOD -->
<%= javascript_importmap_tags @embed_response.viewer.importmap if @embed_response.viewer.importmap %>
<%= stylesheet_link_tag @embed_response.viewer.stylesheet %>
<%= stylesheet_link_tag 'sul_icons.css' %>
Expand Down
2 changes: 1 addition & 1 deletion app/views/layouts/preview/geo.html.erb
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@
crossorigin="anonymous"></script>
<%= stylesheet_link_tag 'geo.css' %>
<%= stylesheet_link_tag 'sul_icons.css' %>
<%#= javascript_importmap_tags 'document' %>
<%= javascript_importmap_tags 'geo' %>
</head>
<body>
<%= yield %>
Expand Down
3 changes: 3 additions & 0 deletions config/importmap.rb
Original file line number Diff line number Diff line change
Expand Up @@ -5,6 +5,7 @@
pin "file", preload: true
pin "document", preload: true
pin "3d", preload: true
pin "geo", preload: true

pin "@hotwired/stimulus", to: "stimulus.min.js", preload: true
pin "@hotwired/stimulus-loading", to: "stimulus-loading.js", preload: true
Expand Down Expand Up @@ -40,3 +41,5 @@
pin "fscreen", to: "https://ga.jspm.io/npm:fscreen@1.2.0/dist/fscreen.cjs.js"
pin "@google/model-viewer", to: "https://cdn.jsdelivr.net/npm/@google/model-viewer@3.5.0/dist/model-viewer-module.min.js"
pin "three", to: "https://ga.jspm.io/npm:three@0.163.0/build/three.module.js"
pin "leaflet" # @1.9.4
pin "Leaflet.Control.Custom"
File renamed without changes.
8 changes: 8 additions & 0 deletions vendor/javascript/leaflet.js

Large diffs are not rendered by default.

0 comments on commit 7117795

Please sign in to comment.