lookbook 2.0.0.beta.1 → 2.0.0.beta.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 03b89a2cdb1b56200c40166eb670887c79a01cf98152653ece5c512fff450492
-  data.tar.gz: ff7da5b63b5023bb3e9f0149af2a49eab5392450cc8851dd258eb7b2973d3267
+  metadata.gz: a6a6aedff44436d4f892bc5b0fc28f596d0cd14f4578af150ea004997f1aa01c
+  data.tar.gz: 548938fd0815d99535f3c70831ad9b5a554c23dd8dd6f6d9896538150b1a79ad
 SHA512:
-  metadata.gz: bac766446880b1f66ac4e68746b638934d1c451e32231a1f24b48b931823ffd11a83147eadeb5106a39c9302759a7f234daba0a2ffbbf7ec46cc9d38090a6757
-  data.tar.gz: 25acb93d4fb260485627def8d01fa7975034662fdc9ca6db621445cf83708c66423e5318655130189bdd1b5edd8f6132e8126658bddc48ad3442bedc1430799f
+  metadata.gz: a3ff63b130c8a92fc6a0f84296d3d56529941a21957eb1ec056d31121ef3eddef308a5397c032d77bf630b9d465085a3dd5f9dd81381abc2fdf88905bfb24ab2
+  data.tar.gz: 8fd98d1823d8a9f5b00d690c73416f98993d8dacf6d8af92935e85891a1c9c06452f82cef88e57c48148bc4329104d2f7583a683ae4e7a5dfb58735d337c1fab

data/app/components/lookbook/embed_code_dropdown/component.html.erb

@@ -1,7 +1,10 @@
 <%= render_component_tag class:"p-3 w-[320px]" do %>  
   <h4 class="text-[11px] uppercase tracking-wider mb-2 font-bold">Preview embed code</h4>
 
-  <p class="text-xs text-gray-600 mb-3">This code can be used to embed this preview in Lookbook pages or on external sites.</p>
+  <p class="text-xs text-gray-600 mb-3">
+    This code can be used to embed this preview in
+    Lookbook pages<% unless policy == "SAMEORIGIN" %> or on external sites<% end %>.
+  </p>
 
   <div class="border-t border-lookbook-dropdown-divider pt-3 pb-3">
     <%= code :html do %><%= embed_code %><% end %>

data/app/components/lookbook/embed_code_dropdown/component.rb

@@ -1,11 +1,12 @@
 module Lookbook
   class EmbedCodeDropdown::Component < Lookbook::BaseComponent
-    attr_reader :preview, :pages, :params, :target
+    attr_reader :preview, :pages, :params, :target, :policy
 
-    def initialize(preview:, target:, pages:, params:, **html_attrs)
+    def initialize(preview:, target:, pages:, params:, policy:, **html_attrs)
       @preview = preview
       @target = target
       @pages = pages
+      @policy = policy
       @params = params.deep_symbolize_keys
       super(**html_attrs)
     end

data/app/controllers/concerns/lookbook/targetable_concern.rb

@@ -42,14 +42,15 @@ module Lookbook
           display_params = SearchParamParser.call(params[:_display])
           display_params.each do |name, value|
             if @dynamic_display_options.key?(name)
-              cookies["lookbook-display-#{name}"] = value
+              cookies["lookbook-display-#{name}"] = value.is_a?(Array) ? value[1] : value
             end
           end
         end
 
         @dynamic_display_options.each do |name, opts|
           choices = opts.is_a?(Hash) ? opts[:choices].to_a : opts
-          @static_display_options[name] ||= cookies.fetch("lookbook-display-#{name}", choices.first)
+          value = choices.first.is_a?(Array) ? choices.first[1] : choices.first
+          @static_display_options[name] ||= cookies.fetch("lookbook-display-#{name}", value)
         end
 
         unless params[:_display]

data/app/controllers/lookbook/previews_controller.rb

@@ -40,7 +40,7 @@ module Lookbook
       if @target
         begin
           opts = {layout: @preview.layout}
-          if params[:lookbook_embed] == "true"
+          if embedded?
             opts[:append_html] = render_to_string("lookbook/partials/_iframe_content_scripts", layout: nil)
           end
           @preview_html = preview_controller.process(:render_in_layout_to_string, "lookbook/previews/group", inspector_data, **opts)
@@ -56,6 +56,10 @@ module Lookbook
 
     private
 
+    def embedded?
+      params[:lookbook_embed] == "true"
+    end
+
     def scenario_json(scenario)
       {
         name: scenario.name,
@@ -65,6 +69,7 @@ module Lookbook
     end
 
     def permit_framing
+      headers["X-Frame-Options"] = Lookbook.config.preview_embeds.policy if embedded?
       headers["X-Frame-Options"] = "SAMEORIGIN" if headers["X-Frame-Options"] == "DENY"
     end
   end

data/app/views/layouts/lookbook/application.html.erb

@@ -29,7 +29,7 @@
           "@click.outside": "closeMobileSidebar",
           cloak: true do %>
 
-          <% cache do %>
+          <% cache @engine.last_changed do %>
 
             <%= lookbook_render :split_layout,
               alpine_data: "$store.layout.#{@pages.any? && @previews.any? ? "sidebar" : "singleSectionSidebar"}",

data/app/views/lookbook/inspector/show.html.erb

@@ -33,13 +33,16 @@
 
         <% toolbar.with_section divide: :left, class: "flex-none relative z-10" do %>
           <%= lookbook_render :button_group do |group| %>
-            <%= group.with_button id: "embed-generator-dropdown-button", icon: :code_2, tooltip: "Get embed code" do |button| %>
-              <% button.with_dropdown.with_content(lookbook_render :embed_code_dropdown,
-                pages: @pages,
-                preview: @preview,
-                target: @target,
-                params: request.query_parameters
-              ) %>
+            <% if @engine.preview_embeds_allowed? %>
+              <%= group.with_button id: "embed-generator-dropdown-button", icon: :code_2, tooltip: "Get embed code" do |button| %>
+                <% button.with_dropdown.with_content(lookbook_render :embed_code_dropdown,
+                  pages: @pages,
+                  preview: @preview,
+                  target: @target,
+                  params: request.query_parameters,
+                  policy: @config.preview_embeds.policy
+                ) %>
+              <% end %>
             <% end %>
             <% group.with_button id: "copy-preview-url-button",
               icon: :link,

data/config/app.yml

@@ -9,7 +9,8 @@ shared:
     main_panels: [preview, output]
     drawer_panels: [source, notes, params, "*"]
   preview_embeds:
-    policy: "SAMEORIGIN"
+    enabled: true
+    policy: SAMEORIGIN
     panels: false
     actions: []
     scenarios: []

data/config/routes.rb

@@ -13,6 +13,8 @@ Lookbook::Engine.routes.draw do
 
   get "/inspect/*path", to: "inspector#show", as: :lookbook_inspect
 
-  get "/embed", to: "embeds#lookup", as: :lookbook_embed_lookup
-  get "/embed/*path", to: "embeds#show", as: :lookbook_embed
+  if Lookbook::Engine.preview_embeds_allowed?
+    get "/embed", to: "embeds#lookup", as: :lookbook_embed_lookup
+    get "/embed/*path", to: "embeds#show", as: :lookbook_embed
+  end
 end

data/lib/lookbook/engine.rb

@@ -82,6 +82,10 @@ module Lookbook
         reloading? && runtime_context.web? && FileWatcher.evented?
       end
 
+      def preview_embeds_allowed?
+        opts.preview_embeds.enabled == true && opts.preview_embeds.policy != "DENY"
+      end
+
       def websocket
         @_websocket ||= auto_refresh? ? Websocket.new(mount_path, logger: Lookbook.logger) : NullWebsocket.new
       end
@@ -150,7 +154,7 @@ module Lookbook
       def preview_watch_paths
         return @_preview_watch_paths if @_preview_watch_paths
 
-        paths = [*opts.preview_paths, opts.components_path, *opts.listen_paths, *view_paths].uniq
+        paths = [*opts.preview_paths, *opts.component_paths, *opts.listen_paths, *view_paths].uniq
         @_preview_watch_paths ||= PathUtils.normalize_paths(paths)
       end
 
@@ -173,11 +177,13 @@ module Lookbook
         changed_files = [*changes[:added], *changes[:modified]] if changes
         parser.parse(changed_files) do |code_objects|
           previews.load(code_objects.all(:class), changes)
+          mark_changed
         end
       end
 
       def load_pages(changes = nil)
         pages.load(Engine.page_paths, changes)
+        mark_changed
       end
 
       def notify_clients(changes = nil)
@@ -192,6 +198,15 @@ module Lookbook
         reloaders.register_changes(changes)
         notify_clients(changes)
       end
+
+      def mark_changed
+        @_last_changed = (Time.now.to_f * 1000).to_i
+      end
+
+      def last_changed
+        mark_changed unless @_last_changed
+        @_last_changed
+      end
     end
 
     at_exit do

data/lib/lookbook/entities/scenario_group_entity.rb

@@ -24,7 +24,7 @@ module Lookbook
     end
 
     def render_targets
-      @_render_targets ||= RenderTargetCollection.new(scenarios.flat_map(&:render_targets).uniq(&:path))
+      @_render_targets ||= RenderTargetCollection.new(scenarios.flat_map(&:render_targets).uniq(&:lookup_path))
     end
 
     def search_terms

data/lib/lookbook/helpers/preview_helper.rb

@@ -10,5 +10,9 @@ module Lookbook
     def lookbook_data(key, fallback = nil)
       Lookbook.data.fetch(key.to_sym, fallback)
     end
+
+    def url_for(*args)
+      main_app.url_for(*args)
+    end
   end
 end

data/lib/lookbook/version.rb

@@ -1,3 +1,3 @@
 module Lookbook
-  VERSION = "2.0.0.beta.1"
+  VERSION = "2.0.0.beta.2"
 end