Polish asset page and drop leftover frozen comments
Address review: remove the non-functional ⋯ actions button and the
redundant "Asset" eyebrow (it sat right under the "Assets" breadcrumb),
and delete the now-unused dots icon.
After rebasing onto main (which removed frozen_string_literal comments
from lib/ and test/), strip the comment from the files this branch adds
— asset.rb, the sole extension, and test_sole.rb. Migrations keep theirs,
matching the rest of db/migrate/.
Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01N92HLTepkg3AtRpDUyZ54o
diff --git a/lib/sequel/extensions/sole.rb b/lib/sequel/extensions/sole.rb
index bab330b..e64c25d 100644
--- a/lib/sequel/extensions/sole.rb
+++ b/lib/sequel/extensions/sole.rb
@@ -1,5 +1,3 @@
-# frozen_string_literal: true
-
require "sequel"
module Sequel
diff --git a/lib/views/asset.rb b/lib/views/asset.rb
index 4db4359..ab4acd5 100644
--- a/lib/views/asset.rb
+++ b/lib/views/asset.rb
@@ -1,5 +1,3 @@
-# frozen_string_literal: true
-
require "phlex"
require_relative "layout"
require_relative "icons"
@@ -49,15 +47,11 @@ module Domus
div(class: "crumbs") do
a(href: "/") { icon("chev-left"); plain "Assets" }
end
- div(class: "asset-actions") do
- button(type: "button", class: "iconbtn", "aria-label": "Asset actions") { icon("dots") }
- end
end
end
def head
header(class: "asset-head") do
- p(class: "eyebrow") { plain "Asset" }
h1(class: "title") { plain @asset[:name] }
tags
end
diff --git a/public/app.css b/public/app.css
index 3736578..5ef54aa 100644
--- a/public/app.css
+++ b/public/app.css
@@ -558,7 +558,6 @@ body {
align-items: center;
margin-bottom: var(--space-s);
}
-.asset-actions { margin-left: auto; }
.asset .crumbs { display: flex; align-items: center; gap: var(--space-2xs); }
.asset .crumbs a {
@@ -575,35 +574,9 @@ body {
.asset .crumbs a:hover { color: var(--w-accent-ink); }
.asset .crumbs svg { width: 13px; height: 13px; }
-.asset .iconbtn {
- display: inline-flex;
- align-items: center;
- justify-content: center;
- width: 36px; height: 36px;
- border: 1px solid var(--w-line-2);
- border-radius: calc(var(--radius) - 5px);
- background: var(--w-surface);
- color: var(--w-ink);
- cursor: pointer;
- transition: background .14s ease;
-}
-.asset .iconbtn:hover { background: var(--w-fill); }
-.asset .iconbtn svg { width: 16px; height: 16px; }
-
-/* ---- title block: catalog eyebrow + name + tags ---- */
+/* ---- title block: name + tags ---- */
.asset-head { margin-bottom: var(--space-s); }
-.asset-head .eyebrow {
- margin: 0 0 var(--space-2xs);
- font-family: var(--font-mono);
- font-size: var(--step--2);
- font-weight: 500;
- line-height: 1;
- text-transform: uppercase;
- letter-spacing: 0.16em;
- color: var(--w-ink-3);
-}
-
.asset-head .title {
margin: 0;
font-family: var(--font-serif);
diff --git a/public/icons/dots.svg b/public/icons/dots.svg
deleted file mode 100644
index 807f5e9..0000000
--- a/public/icons/dots.svg
+++ /dev/null
@@ -1,5 +0,0 @@
-<svg xmlns="http://www.w3.org/2000/svg" width="18" height="18" viewBox="0 0 24 24" fill="currentColor" stroke="none" aria-hidden="true">
- <circle cx="5" cy="12" r="1.5"/>
- <circle cx="12" cy="12" r="1.5"/>
- <circle cx="19" cy="12" r="1.5"/>
-</svg>
diff --git a/test/test_app.rb b/test/test_app.rb
index 4d70625..be61a3d 100644
--- a/test/test_app.rb
+++ b/test/test_app.rb
@@ -69,7 +69,7 @@ class TestApp < Minitest::Test
assert_equal 200, last_response.status
body = last_response.body
assert_includes body, "Bosch 800 dishwasher"
- assert_includes body, "Asset"
+ assert_includes body, "Assets" # breadcrumb back to the index
end
def test_asset_detail_renders_description
diff --git a/test/test_sole.rb b/test/test_sole.rb
index e63aae5..e429357 100644
--- a/test/test_sole.rb
+++ b/test/test_sole.rb
@@ -1,5 +1,3 @@
-# frozen_string_literal: true
-
require_relative "test_helper"
class TestSole < Minitest::Test