More layout adjustments and more dry refactor

app/views/categories/_form.html.erb

@@ -1,5 +1,13 @@
-<!-- app/views/categories/_form.html.erb -->
-<%= form_with(model: category, class: "space-y-6") do |form| %>
+<%= form_with(model: category, class: "space-y-6 max-w-4xl mx-auto bg-white border border-gray-200 rounded-xl shadow-sm p-6 md:p-8") do |form| %>
+  
+  <%= render "layouts/form_header", 
+              model: @category,
+              new_title: "Kategorie anlegen",
+              new_description: "Definiere einen neuen Hardware-Typ für deine Bestandsliste.",
+              edit_title: "Kategorie bearbeiten",
+              edit_description: "Aktualisiere die Bezeichnung oder die Notizen dieser Kategorie." %>
+
+  <hr class="border-gray-200">
 
   <%= render "layouts/form_errors", model: category %>
 

app/views/categories/edit.html.erb

@@ -1,14 +1,21 @@
 <% content_for :title, "Kategorie bearbeiten" %>
 
-<div class="p-4 md:p-6 space-y-6">
+<% content_for :top_bar_actions do %>
+  <%= link_to categories_path, class: "py-2 px-3 text-sm font-medium text-gray-700 bg-white border border-gray-300 rounded-lg hover:bg-gray-50 flex items-center gap-1.5 shadow-sm transition" do %>
+    <svg class="h-4 w-4" fill="none" viewBox="0 0 24 24" stroke-width="2" stroke="currentColor"><path stroke-linecap="round" stroke-linejoin="round" d="M10.5 19.5L3 12m0 0l7.5-7.5M3 12h18" /></svg>
+    <div class="hidden md:inline">
+      Zurück zur Übersicht
+    </div>
+  <% end %>
+<% end %>
+
+<div class="space-y-6 p-4 md:p-6">
   <%= render "form", category: @category %>
 
-  <!-- Gefahrenbereich: Kategorie löschen (Nur wenn keine Items drin sind) -->
-  <div class="max-w-2xl mx-auto bg-red-50/50 border border-red-200 rounded-xl p-6 flex flex-col sm:flex-row justify-between items-start sm:items-center gap-4">
-    <div>
-      <h3 class="text-sm font-bold text-red-800">Kategorie löschen</h3>
-      <p class="text-xs text-red-600 mt-0.5">Dies kann nicht rückgängig gemacht werden. Nur möglich, wenn die Kategorie komplett leer ist.</p>
-    </div>
-    <%= link_to "Löschen", @category, data: { turbo_method: :delete, turbo_confirm: "Möchtest du diese Kategorie wirklich unwiderruflich löschen?" }, class: "py-2 px-3 text-xs font-semibold text-white bg-red-600 hover:bg-red-700 rounded-lg shadow-sm transition" %>
-  </div>
+  <%= render "layouts/danger_zone", 
+            title: "Kategorie löschen",
+            description: "Das Löschen einer Kategorie entfernt diesen Typ dauerhaft. Alle zugeordneten Artikel müssen danach neu kategorisiert werden.",
+            button_text: "Kategorie löschen",
+            confirm_message: "Möchtest du diese Kategorie wirklich löschen? Alle Artikel dieses Typs verlieren ihre Kategorie.",
+            path: category_path(@category) %>
 </div>

app/views/categories/new.html.erb

@@ -1,7 +1,7 @@
 <% content_for :title, "Kategorien anlegen" %>
 
 <% content_for :top_bar_actions do %>
-  <%= link_to rooms_path, class: "py-2 px-3 text-sm font-medium text-gray-700 bg-white border border-gray-300 rounded-lg hover:bg-gray-50 flex items-center gap-1.5 shadow-sm transition" do %>
+  <%= link_to categories_path, class: "py-2 px-3 text-sm font-medium text-gray-700 bg-white border border-gray-300 rounded-lg hover:bg-gray-50 flex items-center gap-1.5 shadow-sm transition" do %>
     <svg class="h-4 w-4" fill="none" viewBox="0 0 24 24" stroke-width="2" stroke="currentColor"><path stroke-linecap="round" stroke-linejoin="round" d="M10.5 19.5L3 12m0 0l7.5-7.5M3 12h18" /></svg>
     <div class="hidden md:inline">
       Zurück zur Übersicht
@@ -9,22 +9,7 @@
   <% end %>
 <% end %>
 
-<div class="max-w-xl mx-auto mt-4">
-  <div class="bg-white border border-gray-200 rounded-xl shadow-sm p-4 md:p-6">
-    
-    <div class="flex items-center gap-3 mb-6 pb-4 border-b border-gray-100 text-gray-500">
-      <div class="p-2 bg-blue-50 text-blue-600 rounded-lg border border-blue-100">
-        <svg class="h-5 w-5" fill="none" viewBox="0 0 24 24" stroke-width="1.5" stroke="currentColor">
-          <path stroke-linecap="round" stroke-linejoin="round" d="M12 4.5v15m7.5-7.5h-15" />
-        </svg>
-      </div>
-      <div>
-        <h3 class="font-bold text-gray-900 text-base">Kategorie erstellen</h3>
-        <p class="text-xs text-gray-400">Definiere einen neuen Hardware-Typ für deine Bestandsliste.</p>
-      </div>
-    </div>
-
-    <%= render "form", category: @category %>
-
-  </div>
+<div class="p-4 md:p-6">
+  <%= render "form", category: @category %>
 </div>
+

app/views/items/_form.html.erb

@@ -1,18 +1,16 @@
-<%= form_with(model: item, class: "space-y-6 max-w-2xl mx-auto bg-white border border-gray-200 rounded-xl shadow-sm p-6 md:p-8") do |form| %>
+<%= form_with(model: item, class: "space-y-6 max-w-4xl mx-auto bg-white border border-gray-200 rounded-xl shadow-sm p-6 md:p-8") do |form| %>
 
-  <%= render "layouts/form_errors", model: item %>
-
-  <div>
-    <h2 class="text-xl font-bold text-gray-800">
-      <%= item.new_record? ? "Artikel registrieren" : "Artikel bearbeiten" %>
-    </h2>
-    <p class="text-sm text-gray-500 mt-1">
-      <%= item.new_record? ? "Trage hier die Unikat-Daten wie Seriennummer und Sticker-ID ein." : "Aktualisiere die Daten oder verändere den aktuellen Standort des Geräts." %>
-    </p>
-  </div>
+ <%= render "layouts/form_header", 
+           model: @item,
+           new_title: "Artikel anlegen",
+           new_description: "Erfasse ein neues Gerät mitsamt SKU, Seriennummer und Sticker-ID.",
+           edit_title: "Artikel bearbeiten",
+           edit_description: "Aktualisiere die Gerätedaten, den Anschaffungspreis oder den Zustand." %>
 
   <hr class="border-gray-200">
 
+  <%= render "layouts/form_errors", model: item %>
+
   <!-- Stammdaten (Name und SKU) -->
   <div class="grid grid-cols-1 sm:grid-cols-2 gap-4">
     <div>

app/views/items/edit.html.erb

@@ -1,16 +1,4 @@
-<!--<%# content_for :title, "Editing item" %>
-
-<div class="md:w-2/3 w-full">
-  <h1 class="font-bold text-4xl">Editing item</h1>
-
-  <%#= render "form", item: @item %>
-
-  <%= link_to "Show this item", @item, class: "w-full sm:w-auto text-center mt-2 sm:mt-0 sm:ml-2 rounded-md px-3.5 py-2.5 bg-gray-100 hover:bg-gray-50 inline-block font-medium" %>
-  <%= link_to "Back to items", items_path, class: "w-full sm:w-auto text-center mt-2 sm:mt-0 sm:ml-2 rounded-md px-3.5 py-2.5 bg-gray-100 hover:bg-gray-50 inline-block font-medium" %>
-</div>
--->
-
-<% content_for :title, "Artikel bearbeiten: #{@item.name}" %>
+<% content_for :title, "Artikel bearbeiten" %>
 
 <!-- OBERE AKTIONSLISTE (Zurück-Button in der Top-Bar) -->
 <% content_for :top_bar_actions do %>
@@ -23,28 +11,14 @@
 
 <div class="space-y-6">
   
-  <!-- Das zentrale Formular mit dem aktuellen Artikel-Objekt laden -->
   <%= render "form", item: @item %>
 
-  <!-- GEFAHRENBEREICH: Artikel löschen -->
-  <div class="max-w-2xl mx-auto bg-red-50/50 border border-red-200 rounded-xl p-6 flex flex-col sm:flex-row justify-between items-start sm:items-center gap-4 shadow-sm">
-    <div class="flex items-start gap-3">
-      <div class="p-2 bg-red-100 text-red-700 rounded-lg shrink-0 mt-0.5">
-        <!-- Heroicon: trash -->
-        <svg class="h-5 w-5" fill="none" viewBox="0 0 24 24" stroke-width="1.5" stroke="currentColor"><path stroke-linecap="round" stroke-linejoin="round" d="m14.74 9-.346 9m-4.788 0L9.26 9m9.968-3.21c.342.052.682.107 1.022.166m-1.022-.165L18.16 19.673a2.25 2.25 0 0 1-2.244 2.077H8.084a2.25 2.25 0 0 1-2.244-2.077L4.772 5.79m14.456 0a48.108 48.108 0 0 0-3.478-.397m-12 .562c.34-.059.68-.114 1.022-.165m0 0a48.11 48.11 0 0 1 3.478-.397m7.5 0v-.916c0-1.18-.91-2.164-2.09-2.201a51.964 51.964 0 0 0-3.32 0c-1.18.037-2.09 1.022-2.09 2.201v.916m7.5 0a48.667 48.667 0 0 0-7.5 0" /></svg>
-      </div>
-      <div>
-        <h3 class="text-sm font-bold text-red-800">Artikel dauerhaft ausbuchen</h3>
-        <p class="text-xs text-red-600 mt-0.5">Dies entfernt das Gerät unwiderruflich aus dem Bestand. Die bisherige Verlaufshistorie wird dabei ebenfalls gelöscht.</p>
-      </div>
-    </div>
-    
-    <!-- Löschen-Link gekoppelt an Rails Turbo-Method-Delete -->
-    <%= link_to "Artikel löschen", 
-                item_path(@item), 
-                data: { turbo_method: :delete, turbo_confirm: "Möchtest du diesen Artikel wirklich unwiderruflich aus dem System entfernen?" }, 
-                class: "py-2 px-4 text-sm font-semibold text-white bg-red-600 hover:bg-red-700 rounded-lg shadow-sm transition" %>
-  </div>
+  <%= render "layouts/danger_zone", 
+            title: "Artikel restlos löschen",
+            description: "Dieser Artikel wird dauerhaft und unwiderruflich aus der Bestandsliste entfernt. Auch die QR-Code-Zuordnung erlischt.",
+            button_text: "Artikel löschen",
+            confirm_message: "Möchtest du diesen Artikel wirklich permanent aus dem System entfernen?",
+            path: item_path(@item) %>
 
 </div>
 

app/views/items/new.html.erb

@@ -1,15 +1,4 @@
-<!-- <%# content_for :title, "New item" %>
-
-<div class="md:w-2/3 w-full">
-  <h1 class="font-bold text-4xl">New item</h1>
-
-  <%#= render "form", item: @item %>
-
-  <%= link_to "Back to items", items_path, class: "w-full sm:w-auto text-center mt-2 sm:mt-0 sm:ml-2 rounded-md px-3.5 py-2.5 bg-gray-100 hover:bg-gray-50 inline-block font-medium" %>
-</div>
--->
-
-<% content_for :title, "Neuen Artikel hinzufügen" %>
+<% content_for :title, "Artikel anlegen" %>
 
 <!-- OBERE LEISTE (Zurück-Button in der Top-Bar via Layout-Yield) -->
 <% content_for :top_bar_actions do %>
@@ -21,7 +10,6 @@
 <% end %>
 
 <div class="p-4 md:p-6">
-  <!-- Lädt das Formular-Partial und übergibt das leere Artikel-Objekt -->
   <%= render "form", item: @item %>
 </div>
 

app/views/layouts/_danger_zone.html.erb

@@ -0,0 +1,20 @@
+<div class="max-w-4xl mx-auto bg-red-50/50 border border-red-200 rounded-xl p-6 flex flex-col sm:flex-row justify-between items-start sm:items-center gap-4 shadow-sm mt-8">
+  <div class="flex items-start gap-3">
+    <!-- Mülleimer-Icon -->
+    <div class="p-2 bg-red-100 text-red-700 rounded-lg shrink-0 mt-0.5">
+      <svg class="h-5 w-5" fill="none" viewBox="0 0 24 24" stroke-width="1.5" stroke="currentColor">
+        <path stroke-linecap="round" stroke-linejoin="round" d="m14.74 9-.346 9m-4.788 0L9.26 9m9.968-3.21c.342.052.682.107 1.022.166m-1.022-.165L18.16 19.673a2.25 2.25 0 0 1-2.244 2.077H8.084a2.25 2.25 0 0 1-2.244-2.077L4.772 5.79m14.456 0a48.108 48.108 0 0 0-3.478-.397m-12 .562c.34-.059.68-.114 1.022-.165m0 0a48.11 48.11 0 0 1 3.478-.397m7.5 0v-.916c0-1.18-.91-2.164-2.09-2.201a51.964 51.964 0 0 0-3.32 0c-1.18.037-2.09 1.022-2.09 2.201v.916m7.5 0a48.667 48.667 0 0 0-7.5 0" />
+      </svg>
+    </div>
+    <div class="min-w-0">
+      <h3 class="text-sm font-bold text-red-800"><%= title %></h3>
+      <p class="text-xs text-red-600 mt-0.5 leading-relaxed"><%= description %></p>
+    </div>
+  </div>
+  
+  <!-- Der dynamische Turbo-Delete-Button -->
+  <%= link_to button_text, 
+              path, 
+              data: { turbo_method: :delete, turbo_confirm: confirm_message }, 
+              class: "py-2 px-4 text-sm font-semibold text-white bg-red-600 hover:bg-red-700 rounded-lg shadow-sm transition whitespace-nowrap shrink-0" %>
+</div>

app/views/layouts/_form_header.html.erb

@@ -0,0 +1,26 @@
+<%
+  # 1. Wir prüfen automatisch, ob es ein neuer oder bestehender Eintrag ist
+  is_new = model.new_record?
+
+  # 2. Farb- und Icon-Weiche (Blau für Neu, Amber für Bearbeiten)
+  bg_css     = is_new ? "bg-blue-50 border-blue-100 text-blue-600" : "bg-amber-50 border-amber-100 text-amber-600"
+  title_text = is_new ? new_title : edit_title
+  desc_text  = is_new ? new_description : edit_description
+
+  # 3. Die beiden SVGs (Plus vs. Stift) mit identischer Konturschärfe (stroke-width="2")
+  icon_plus = '<svg class="h-5 w-5" fill="none" viewBox="0 0 24 24" stroke-width="2" stroke="currentColor"><path stroke-linecap="round" stroke-linejoin="round" d="M12 4.5v15m7.5-7.5h-15" /></svg>'
+  icon_edit = '<svg class="h-5 w-5" fill="none" viewBox="0 0 24 24" stroke-width="2" stroke="currentColor"><path stroke-linecap="round" stroke-linejoin="round" d="m16.862 4.487 1.687-1.688a1.875 1.875 0 1 1 2.652 2.652L10.582 16.07a4.5 4.5 0 0 1-1.897 1.13L6 18l.8-2.685a4.5 4.5 0 0 1 1.13-1.897l8.932-8.931Zm0 0L19.5 7.125M18 14v4.75A2.25 2.25 0 0 1 15.75 21H5.25A2.25 2.25 0 0 1 3 18.75V8.25A2.25 2.25 0 0 1 5.25 6H10" /></svg>'
+  
+  final_icon = is_new ? icon_plus.html_safe : icon_edit.html_safe
+%>
+
+<div class="flex items-center gap-3">
+  <!-- Das Icon-Quadrat wechselt dynamisch seine Farbe und Grafik -->
+  <div class="p-2 rounded-lg border shrink-0 <%= bg_css %>">
+    <%= final_icon %>
+  </div>
+  <div class="min-w-0">
+    <h2 class="text-xl font-bold text-gray-800 leading-tight"><%= title_text %></h2>
+    <p class="text-sm text-gray-500 mt-1 leading-normal"><%= desc_text %></p>
+  </div>
+</div>

app/views/rooms/_form.html.erb

@@ -1,18 +1,17 @@
-<%= form_with(model: room, class: "space-y-6 max-w-xl mx-auto bg-white border border-gray-200 rounded-xl shadow-sm p-6 md:p-8") do |form| %>
+<%= form_with(model: room, class: "space-y-6 max-w-4xl mx-auto bg-white border border-gray-200 rounded-xl shadow-sm p-6 md:p-8") do |form| %>
 
-  <%= render "layouts/form_errors", model: room %>
 
-  <div>
-    <h2 class="text-xl font-bold text-gray-800">
-      <%= room.new_record? ? "Raum anlegen" : "Raum bearbeiten" %>
-    </h2>
-    <p class="text-sm text-gray-500 mt-1">
-      <%= room.new_record? ? "Definiere einen neuen physischen Standort für deine Inventargegenstände." : "Aktualisiere die Standortbezeichnung oder die Etage." %>
-    </p>
-  </div>
+  <%= render "layouts/form_header", 
+           model: @room,
+           new_title: "Raum anlegen",
+           new_description: "Definiere einen neuen physischen Standort für deine Inventargegenstände.",
+           edit_title: "Raum bearbeiten",
+           edit_description: "Aktualisiere die Raumbezeichnung, das Gebäude oder die Etage." %>
 
   <hr class="border-gray-200">
   
+  <%= render "layouts/form_errors", model: room %>
+
   <!-- Raumnummer / Name -->
   <div>
     <%= form.label :name, "Raumbezeichnung / Raumnummer", class: "block text-sm font-medium mb-1.5 text-gray-700" %>

app/views/rooms/edit.html.erb

@@ -3,25 +3,19 @@
 <% content_for :top_bar_actions do %>
   <%= link_to rooms_path, class: "py-2 px-3 text-sm font-medium text-gray-700 bg-white border border-gray-300 rounded-lg hover:bg-gray-50 flex items-center gap-1.5 shadow-sm transition" do %>
     <svg class="h-4 w-4" fill="none" viewBox="0 0 24 24" stroke-width="2" stroke="currentColor"><path stroke-linecap="round" stroke-linejoin="round" d="M10.5 19.5L3 12m0 0l7.5-7.5M3 12h18" /></svg>
-    Zurück zur Übersicht
+    <div class="hidden md:inline">
+      Zurück zur Übersicht
+    </div>
   <% end %>
 <% end %>
 
 <div class="space-y-6 p-4 md:p-6">
   <%= render "form", room: @room %>
 
-  <!-- GEFAHRENBEREICH: Raum löschen -->
-  <div class="max-w-xl mx-auto bg-red-50/50 border border-red-200 rounded-xl p-6 flex flex-col sm:flex-row justify-between items-start sm:items-center gap-4 shadow-sm">
-    <div class="flex items-start gap-3">
-      <div class="p-2 bg-red-100 text-red-700 rounded-lg shrink-0 mt-0.5">
-        <svg class="h-5 w-5" fill="none" viewBox="0 0 24 24" stroke-width="1.5" stroke="currentColor"><path stroke-linecap="round" stroke-linejoin="round" d="m14.74 9-.346 9m-4.788 0L9.26 9m9.968-3.21c.342.052.682.107 1.022.166m-1.022-.165L18.16 19.673a2.25 2.25 0 0 1-2.244 2.077H8.084a2.25 2.25 0 0 1-2.244-2.077L4.772 5.79m14.456 0a48.108 48.108 0 0 0-3.478-.397m-12 .562c.34-.059.68-.114 1.022-.165m0 0a48.11 48.11 0 0 1 3.478-.397m7.5 0v-.916c0-1.18-.91-2.164-2.09-2.201a51.964 51.964 0 0 0-3.32 0c-1.18.037-2.09 1.022-2.09 2.201v.916m7.5 0a48.667 48.667 0 0 0-7.5 0" /></svg>
-      </div>
-      <div>
-        <h3 class="text-sm font-bold text-red-800">Raum löschen</h3>
-        <p class="text-xs text-red-600 mt-0.5">In diesem Raum gelistete Artikel verlieren ihren Standort und werden automatisch ins Hauptlager umgebucht.</p>
-      </div>
-    </div>
-    
-    <%= link_to "Raum löschen", room_path(@room), data: { turbo_method: :delete, turbo_confirm: "Möchtest du diesen Standort wirklich löschen?" }, class: "py-2 px-4 text-sm font-semibold text-white bg-red-600 hover:bg-red-700 rounded-lg shadow-sm transition" %>
-  </div>
+  <%= render "layouts/danger_zone", 
+            title: "Raum löschen",
+            description: "In diesem Raum gelistete Artikel verlieren ihren Standort und werden automatisch ins Hauptlager umgebucht.",
+            button_text: "Raum löschen",
+            confirm_message: "Möchtest du diesen Standort wirklich löschen?",
+            path: room_path(@room) %>
 </div>