From 836308471f8d31ccdcdd3a5bd88bc76cc1c0831b Mon Sep 17 00:00:00 2001 From: erdgeist Date: Mon, 20 Jul 2026 19:45:43 +0200 Subject: Make headline images explicit, add asset credits - related_assets gains a `headline` boolean (DB-enforced: at most one per page), replacing "first image by position" as the headline rule. A rake task backfills the current first image on every live head/draft, so nothing changes visually until an editor changes it. - The image picker sidebar gets a star toggle reflecting the flag; the TinyMCE inline-image picker's badge now reads it too, instead of assuming position 0. - No headline chosen (or none attached) now falls back to the gallery-count caption itself becoming the lightbox trigger, instead of the gallery being unreachable. - Assets gain creator, source_url, and license_key (against a new config/asset_licenses.yml dictionary). asset_credit renders a degrading attribution line, reused as a hidden per-image glightbox caption so credit is one click away for every image, not only the headline's always-visible one. - Fixed: asset thumbnails rendered unconditionally regardless of whether a real variant exists on disk. Asset#has_variant? checks file existence, not content type -- some legacy PDFs have real pre-rewrite thumbnails a content-type check would have hidden. - assets#new/edit rebuilt onto the same node_description/node_content layout as assets#show, picking up the three new fields in the process. --- app/views/assets/edit.html.erb | 58 +++++++++++++++++++++++++++-------------- app/views/assets/index.html.erb | 2 +- app/views/assets/new.html.erb | 49 ++++++++++++++++++++++++---------- app/views/assets/show.html.erb | 26 +++++++++++++++++- 4 files changed, 100 insertions(+), 35 deletions(-) (limited to 'app/views/assets') diff --git a/app/views/assets/edit.html.erb b/app/views/assets/edit.html.erb index f1986000..3ef8fea5 100644 --- a/app/views/assets/edit.html.erb +++ b/app/views/assets/edit.html.erb @@ -1,24 +1,44 @@ -

Editing asset

+
+

Editing asset

-<%= form_for(@asset, html: { multipart: true }) do |f| %> - <%= form_error_messages(f) %> + <%= form_for(@asset, html: { multipart: true }) do |f| %> + <%= form_error_messages(f) %> - <% if @asset.upload.present? %> -

- Current file: - <%= @asset.upload.url %> - (<%= number_to_human_size(@asset.upload.size) %>) -

- <% end %> +
+
<%= f.label :name %>
+
<%= f.text_field :name %>
+ + <% if @asset.upload.present? %> +
Current File
+
<%= @asset.upload.url %> (<%= number_to_human_size(@asset.upload.size) %>)
+ <% end %> + +
<%= f.label :upload, "Replace file" %>
+
<%= f.file_field :upload %>
-

-
- <%= f.file_field :upload %> -

+
<%= f.label :creator %>
+
<%= f.text_field :creator %>
-

- <%= f.submit 'Update' %> -

-<% end %> +
<%= f.label :source_url, "Source URL" %>
+
<%= f.text_field :source_url %>
-<%= link_to 'Show', @asset %> | <%= link_to 'Back', assets_path %> +
<%= f.label :license_key, "License" %>
+
+ <%= f.select :license_key, + options_for_select( + [["— none —", ""]] + AssetLicense.keys.map { |key| [t("asset_licenses.#{key}"), key] }, + @asset.license_key + ) %> +
+ +
Actions
+
+
+
<%= f.submit 'Update' %>
+
<%= link_to 'Show', @asset %>
+
<%= link_to 'Back', assets_path %>
+
+
+
+ <% end %> +
diff --git a/app/views/assets/index.html.erb b/app/views/assets/index.html.erb index 8c355617..ff1300a2 100644 --- a/app/views/assets/index.html.erb +++ b/app/views/assets/index.html.erb @@ -17,7 +17,7 @@ <% @assets.each do |asset| %> - <%= image_tag asset.upload.url(:thumb), style: "max-width: 100px; max-height: 100px;" %> + <% if asset.has_variant?(:thumb) %><%= image_tag asset.upload.url(:thumb), style: "max-width: 100px; max-height: 100px;" %><% end %> <%= link_to asset.name, asset.upload.url %> <%= asset.upload.content_type %> <%= link_to 'Show', asset %> diff --git a/app/views/assets/new.html.erb b/app/views/assets/new.html.erb index 6c1310a7..2cf88650 100644 --- a/app/views/assets/new.html.erb +++ b/app/views/assets/new.html.erb @@ -1,17 +1,38 @@ -

New asset

+
+

New asset

-<%= form_for(@asset, :html => { :multipart => true }) do |f| %> - <%= form_error_messages(f) %> - -

- <%= f.label :name %>
- <%= f.text_field :name %> -

+ <%= form_for(@asset, :html => { :multipart => true }) do |f| %> + <%= form_error_messages(f) %> -

- <%= f.file_field :upload %> - <%= f.submit 'Create' %> -

-<% end %> +
+
<%= f.label :name %>
+
<%= f.text_field :name %>
-<%= link_to 'Back', assets_path %> +
<%= f.label :upload, "File" %>
+
<%= f.file_field :upload %>
+ +
<%= f.label :creator %>
+
<%= f.text_field :creator %>
+ +
<%= f.label :source_url, "Source URL" %>
+
<%= f.text_field :source_url %>
+ +
<%= f.label :license_key, "License" %>
+
+ <%= f.select :license_key, + options_for_select( + [["— none —", ""]] + AssetLicense.keys.map { |key| [t("asset_licenses.#{key}"), key] }, + @asset.license_key + ) %> +
+ +
Actions
+
+
+
<%= f.submit 'Create' %>
+
<%= link_to 'Back', assets_path %>
+
+
+
+ <% end %> +
diff --git a/app/views/assets/show.html.erb b/app/views/assets/show.html.erb index ff00883c..a0e4e464 100644 --- a/app/views/assets/show.html.erb +++ b/app/views/assets/show.html.erb @@ -17,7 +17,10 @@
Thumbnail
-
<%= image_tag @asset.upload.url(:medium), style: "max-width: 300px; max-height: 300px;" %>
+ <% if @asset.has_variant?(:medium) %> +
Thumbnail
+
<%= image_tag @asset.upload.url(:medium), style: "max-width: 300px; max-height: 300px;" %>
+ <% end %>
Public Path
@@ -30,6 +33,27 @@
+
Creator
+
<%= @asset.creator.presence || "—" %>
+ +
Source
+
+ <% if @asset.source_url.present? %> + <%= link_to @asset.source_url, @asset.source_url %> + <% else %> + — + <% end %> +
+ +
License
+
+ <% if (license = AssetLicense.find(@asset.license_key)) %> + <%= t("asset_licenses.#{license.key}") %> + <% else %> + — + <% end %> +
+
Content Type
<%= @asset.upload.content_type %>
-- cgit v1.3