jekyll-l10n 1.0.7 → 1.1.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: beb5fa169b7e1fdb683dcf59de3d13e6d9d222f8fc8c2408cb0e779139126a82
-  data.tar.gz: b6b3f4a8a4bc06892581b9017b8018cc5d6c2c0b44f8599e3015fa123b29470f
+  metadata.gz: 89d7ff23e3bc3ebcb0f27aa77665ac410afcb96ebff6b5abb1992349a68b1859
+  data.tar.gz: b88c9b6283535514aebd9a77e2d5fe4f01407f008e5de2b8d21e0b30cb76bc85
 SHA512:
-  metadata.gz: df4c40e8bb987e6eb0b857b56a8b02c90ec4eec3bff830ff706a905f95d82212d4e43629660e34d708dc5b7f034ea193bb026caebe7e1889ffc526165eb154dc
-  data.tar.gz: 63282d1fad6f225a9e34fb39d56eade8a3fe5965e793993d5aded4a41b43bee55fad61b9d8c36ae3d5cd3d63ef964f771e7196b003d1fa632b3460e0fbba6d85
+  metadata.gz: 3fd8a693edcc99826f5f1beff125706b2dab0d39cccf21e393dfc2895578ddfaead57c309b1dbb0ba8cfe4d14c280fab34a33c895f87bc47a86ad972c9a395d2
+  data.tar.gz: 69b96d33ceaf4bcbe8e2ab8073e7d6fe549e1cfd5e2b09cce41ea1f32def1df650c8c7347ff8b15c230c7c237b3073ff77b015c8881b291b2f3a6c50ed020fa1

data/lib/jekyll-l10n/jekyll/generator.rb

@@ -74,8 +74,24 @@ module Jekyll
 
       def should_localize_page?(page)
         return false if page.data["localized"] == true
+        return false unless page.data["with_locales"] == true
+
+        # Only localize HTML pages - exclude static assets
+        # Check multiple conditions to be safe:
+        # 1. output_ext must be .html
+        # 2. path must not contain common asset directories
+        is_html = page.output_ext == ".html"
+        is_not_asset = !page.path.match?(%r!/(assets|vendor|node_modules|\..*)/!i)
+
+        # Debug: Log which pages are being skipped
+        if !is_html
+          Jekyll.logger.debug "Localization",
+                              "Skipping non-HTML: #{page.path} (ext: #{page.output_ext})"
+        elsif !is_not_asset
+          Jekyll.logger.debug "Localization", "Skipping asset: #{page.path}"
+        end
 
-        page.data["with_locales"] == true
+        is_html && is_not_asset
       end
 
       def get_page_locales(page)

data/lib/jekyll-l10n/jekyll/localized_page.rb

@@ -95,28 +95,38 @@ module Jekyll
       #
       # Returns the page's URL with a locale prefix. For example, if the source page URL is
       # '/path/to/page/', this returns '/es/path/to/page/' for locale 'es'.
-      # The URL always ends with a trailing slash for consistency.
+      # For HTML pages, URLs end with a trailing slash. For assets, the file extension is preserved.
       #
-      # @return [String] The locale-prefixed URL (e.g., '/es/path/to/page/')
+      # @return [String] The locale-prefixed URL with optional trailing slash
+      #   (e.g., '/es/path/to/page/' or '/es/assets/style.css')
       def url
         url = "/#{@locale}#{@original_url}"
-        url += "/" unless url.end_with?("/")
+        # Only add trailing slash for directory URLs (no file extension)
+        url += "/" unless url.end_with?("/") || url =~ %r!\.[a-zA-Z0-9]+$!
         url
       end
 
       # Get the destination file path for this localized page
       #
-      # Computes the full file system path where this page's HTML will be written.
-      # For a page with URL '/es/path/to/page/', this returns 'dest/es/path/to/page/index.html'.
+      # Computes the full file system path where this page's content will be written.
+      # For HTML pages with URL '/es/path/to/page/', this returns 'dest/es/path/to/page/index.html'.
+      # For assets with URL '/es/assets/style.css', this returns 'dest/es/assets/style.css'.
       #
       # @param dest [String, nil] The destination directory (defaults to nil, which uses
       #   site config destination)
-      # @return [String] The full file system path for the output HTML file
+      # @return [String] The full file system path for the output file
       def destination(dest = nil)
         dest ||= @site.config["destination"]
         url_path = url
-        url_path += "/" unless url_path.end_with?("/")
-        "#{dest}#{url_path}index.html"
+
+        # Only use index.html for HTML pages
+        if @output_ext == ".html"
+          url_path += "/" unless url_path.end_with?("/")
+          "#{dest}#{url_path}index.html"
+        else
+          # For assets, preserve the file extension
+          "#{dest}#{url_path}"
+        end
       end
 
       # Convert this localized page to Liquid template data

data/lib/jekyll-l10n/utils/url_path_builder.rb

@@ -30,13 +30,22 @@ module Jekyll
 
       # Convert a URL to a file system path.
       #
-      # Converts Jekyll URL to the path it would be written to on disk,
-      # using index.html for directory-based URLs.
+      # Converts Jekyll URL to the path it would be written to on disk.
+      # For URLs with file extensions (assets), preserves the extension.
+      # For directory-style URLs, uses index.html.
       #
-      # @param url [String] Jekyll URL (e.g., '/docs/page.html')
-      # @return [String] File path (e.g., 'docs/page.html/index.html')
+      # @param url [String] Jekyll URL (e.g., '/docs/page.html' or '/assets/style.css')
+      # @return [String] File path (e.g., 'docs/page.html/index.html' or 'assets/style.css')
       def url_to_file_path(url)
-        "#{normalize_url(url)}/index.html"
+        normalized = normalize_url(url)
+
+        # Preserve file extensions for assets (css, js, etc.)
+        # Only add /index.html for directory-style URLs
+        if %r!\.[a-zA-Z0-9]+$!.match?(normalized)
+          normalized
+        else
+          "#{normalized}/index.html"
+        end
       end
 
       # Convert a URL to a PO page path for translation files.

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: jekyll-l10n
 version: !ruby/object:Gem::Version
-  version: 1.0.7
+  version: 1.1.0
 platform: ruby
 authors:
 - ReleaseBot