From 781b2bb9f04d731cbff767e4603020dfde35b2a3 Mon Sep 17 00:00:00 2001
From: Jez Caudle <jez@hiddenagenda.ltd.uk>
Date: Mon, 20 May 2024 11:19:02 +0100
Subject: [PATCH] Actually fixed the confirm: issue. They are link_to now
 Trimmed the CSS.

---
 app/assets/stylesheets/application.css | 10 +++-------
 app/views/domains/index.html.erb       | 15 +++++++++------
 app/views/domains/show.html.erb        |  6 +++---
 3 files changed, 15 insertions(+), 16 deletions(-)

diff --git a/app/assets/stylesheets/application.css b/app/assets/stylesheets/application.css
index 565df94..faf56d5 100644
--- a/app/assets/stylesheets/application.css
+++ b/app/assets/stylesheets/application.css
@@ -105,14 +105,10 @@ menu > li {
 }
 
 .domain-header a {
-  font-size: 9pt;
+  scale: 75%;
 }
 
-.domain-header button {
-  margin:0 0 0 10px;
-}
-
-.email-list {
+.email-list, .domain-list {
   border-radius:1rem;
   border:1rem black;
   background-color:#e7eae7;
@@ -126,7 +122,7 @@ menu > li {
 }
 
 .email-list li a {
-  padding:0 10px;
+  padding:0 0.3rem;
 }
 
 footer {
diff --git a/app/views/domains/index.html.erb b/app/views/domains/index.html.erb
index aa50cbb..6eb4fb8 100644
--- a/app/views/domains/index.html.erb
+++ b/app/views/domains/index.html.erb
@@ -1,11 +1,14 @@
-<h1>Domains <%= link_to "Add new domain",new_domain_path %> </h1>
+<div class="domain">
+  <div class="domain-header">
+    <h2>Domains <%= link_to "Add new domain",new_domain_path %></h2>
+  </div>
 
-<div id="domains">
-  <ul>
-  <% @domains.each do |domain| %>
-    <li><%= link_to domain.domain, domain %></li>
+  <div class="domain-list">
+    <ul><% @domains.each do |domain| %>
+      <li><%= link_to domain.domain, domain %></li>
   <% end %>
-  </ul>
+    </ul>
+  </div>
 </div>
 
 
diff --git a/app/views/domains/show.html.erb b/app/views/domains/show.html.erb
index 82ffa89..fe297c7 100644
--- a/app/views/domains/show.html.erb
+++ b/app/views/domains/show.html.erb
@@ -1,12 +1,12 @@
 <div class="domain">
   <div class="domain-header">
-    <h2><%= @domain.domain %> <%#= link_to "Edit", edit_domain_path(@domain) %> <%= button_to "Delete", @domain, data: { turbo_method: :delete,  turbo_confirm: "Are you sure?"} %></h2>
+    <h2><%= @domain.domain %> <%#= link_to "Edit", edit_domain_path(@domain) %> <%= link_to "Delete", @domain, data: { turbo_method: :delete,  turbo_confirm: "Are you sure?"} %></h2>
   </div>
   <div class="email-list">
     <h3>Credentials (<%= @domain.credentials.count %>) <%= link_to "Add #{@domain.domain} email address", new_domain_credential_path(@domain) %></h3>
       <ul>
         <% @domain.credentials.each do |credential| %>
-        <li><%= credential.email %> <%= link_to "Edit", edit_domain_credential_path(@domain,credential) %> <%= button_to "Delete this email address", [@domain, credential], data: { turbo_method: :delete,  turbo_confirm: "Are you sure?"} %></li>
+        <li><%= credential.email %> <%= link_to "Edit", edit_domain_credential_path(@domain,credential) %> <%= link_to "Delete this email address", [@domain, credential], data: { turbo_method: :delete,  turbo_confirm: "Are you sure?"} %></li>
         <% end %>
       </ul>
   </div>
@@ -15,7 +15,7 @@
     <h3>Virtuals (<%= @domain.virtuals.count %>) <%= link_to "Add #{@domain.domain} virtual email", new_domain_virtual_path(@domain) %></h3>
     <ul>
       <% @domain.virtuals.each do |virtual| %>
-      <li><%= virtual.email%> >> <%= virtual.destination%> <%= link_to "Edit", edit_domain_virtual_path(@domain,virtual)%> <%= button_to "Delete", [@domain, virtual], data: { turbo_method: :delete,  turbo_confirm: "Are you sure?"} %></li>
+      <li><%= virtual.email%> >> <%= virtual.destination%> <%= link_to "Edit", edit_domain_virtual_path(@domain,virtual)%> <%= link_to "Delete", [@domain, virtual], data: { turbo_method: :delete,  turbo_confirm: "Are you sure?"} %></li>
       <% end %>
     </ul>