From 56a60d48f6cc11fb512e249f4bf32bf69e902924 Mon Sep 17 00:00:00 2001
From: Miquel Sabaté Solà <msabate@suse.com>
Date: Thu, 14 Mar 2024 18:05:40 +0100
Subject: Make the UI a bit more consistent
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

- Titles have been unified by using the same HTML tag.
- Centered views have been improved for big screens.
- Added more views that are to be centered.

Signed-off-by: Miquel Sabaté Solà <msabate@suse.com>
---
 app/assets/stylesheets/tags.css          |  4 ++--
 app/assets/stylesheets/utils.css         |  2 +-
 app/views/exports/new.html.erb           | 24 +++++++++++++++---------
 app/views/sessions/new.html.erb          |  2 +-
 app/views/shared_searches/index.html.erb | 24 +++++++++++++++---------
 app/views/tags/index.html.erb            | 16 ++++++++++------
 app/views/tags/new.html.erb              |  2 +-
 7 files changed, 45 insertions(+), 29 deletions(-)

(limited to 'app')

diff --git a/app/assets/stylesheets/tags.css b/app/assets/stylesheets/tags.css
index 141cbca..75dd87d 100644
--- a/app/assets/stylesheets/tags.css
+++ b/app/assets/stylesheets/tags.css
@@ -1,9 +1,9 @@
-#tag-index h2 span {
+#tag-index h3 span {
     font-size: 1rem;
     vertical-align: middle;
 }
 
-#tag-new h2 span {
+#tag-new h3 span {
     font-size: 1rem;
     vertical-align: middle;
 }
diff --git a/app/assets/stylesheets/utils.css b/app/assets/stylesheets/utils.css
index 3659b13..d4bdb4b 100644
--- a/app/assets/stylesheets/utils.css
+++ b/app/assets/stylesheets/utils.css
@@ -4,7 +4,7 @@
 
 .center-contents {
     display: grid;
-    grid-template-columns: repeat(3, 1fr);
+    grid-template-columns: 1fr 2fr 1fr;
 }
 
 @media only screen and (max-width: 720px) {
diff --git a/app/views/exports/new.html.erb b/app/views/exports/new.html.erb
index 0f47bf8..631da20 100644
--- a/app/views/exports/new.html.erb
+++ b/app/views/exports/new.html.erb
@@ -1,14 +1,20 @@
-<div class="notice"><b><%= I18n.t('searches.export.note') %></b>: <%= I18n.t('searches.export.note-msg') %>.</div>
+<div class="center-contents">
+    <div></div>
+    <div>
+        <div class="notice"><b><%= I18n.t('searches.export.note') %></b>: <%= I18n.t('searches.export.note-msg') %>.</div>
 
-<div data-controller="export">
-    <div id="errors"></div>
+        <div data-controller="export">
+            <div id="errors"></div>
 
-    <input type="hidden" data-export-target="id" value="<%= @search.id %>" />
+            <input type="hidden" data-export-target="id" value="<%= @search.id %>" />
 
-    <select id="export-select-format" data-action="export#change">
-        <option selected="selected" value="csv">CSV</option>
-        <option value="uoc">UOC</option>
-    </select>
+            <select id="export-select-format" data-action="export#change">
+                <option selected="selected" value="csv">CSV</option>
+                <option value="uoc">UOC</option>
+            </select>
 
-    <a id="export-format-button" class="button" href="#"><%= I18n.t('searches.export.action').capitalize %></a>
+            <a id="export-format-button" class="button" href="#"><%= I18n.t('searches.export.action').capitalize %></a>
+        </div>
+    </div>
+    <div></div>
 </div>
diff --git a/app/views/sessions/new.html.erb b/app/views/sessions/new.html.erb
index c6d7f4f..dfd4ca9 100644
--- a/app/views/sessions/new.html.erb
+++ b/app/views/sessions/new.html.erb
@@ -1,7 +1,7 @@
 <div class="center-contents">
   <div></div>
   <div>
-    <h2><%= I18n.t('sessions.title') %></h2>
+    <h3><%= I18n.t('sessions.title') %></h3>
 
     <%= form_tag sessions_path do |form| %>
       <div class='field'>
diff --git a/app/views/shared_searches/index.html.erb b/app/views/shared_searches/index.html.erb
index 9a145f6..a17537c 100644
--- a/app/views/shared_searches/index.html.erb
+++ b/app/views/shared_searches/index.html.erb
@@ -1,13 +1,19 @@
-<h3><%= I18n.t('searches.shared.title') %></h3>
+<div class="center-contents">
+    <div></div>
+    <div>
+        <h3><%= I18n.t('searches.shared.title') %></h3>
 
-<% if @searches.empty? %>
-    <div><span><%= I18n.t('searches.shared.none') %>.</span></div>
-<% else %>
-    <div id="shared-searches-list">
-        <% @searches.each do |s| %>
-            <div>
-                <%= link_to s.name, search_shared_path(s) %>
+        <% if @searches.empty? %>
+            <div><span><%= I18n.t('searches.shared.none') %>.</span></div>
+        <% else %>
+            <div id="shared-searches-list">
+                <% @searches.each do |s| %>
+                    <div>
+                        <%= link_to s.name, search_shared_path(s) %>
+                    </div>
+                <% end %>
             </div>
         <% end %>
     </div>
-<% end %>
+    <div></div>
+</div>
diff --git a/app/views/tags/index.html.erb b/app/views/tags/index.html.erb
index 329e069..c91f374 100644
--- a/app/views/tags/index.html.erb
+++ b/app/views/tags/index.html.erb
@@ -1,13 +1,17 @@
 <div class="center-contents">
   <div></div>
   <div id="tag-index">
-    <h2><%= I18n.t('tags.title') %> <span>&#8212; <%= link_to I18n.t('tags.new-action'), new_tag_path %></span></h2>
+    <h3><%= I18n.t('tags.title') %><% if @tags.any? %><span>&#8212; <%= link_to I18n.t('tags.new-action'), new_tag_path %></span><% end %></h3>
 
-    <div id="tags-list">
-      <% @tags.each do |tag| %>
-        <%= render tag %>
-      <% end %>
-    </div>
+    <% if @tags.any? %>
+      <div id="tags-list">
+        <% @tags.each do |tag| %>
+          <%= render tag %>
+        <% end %>
+      </div>
+    <% else %>
+      <div><span><%= I18n.t('tags.none') %> <%= link_to I18n.t('license.here'), new_tag_path %>.</span></div>
+    <% end %>
   </div>
   <div></div>
 </div>
diff --git a/app/views/tags/new.html.erb b/app/views/tags/new.html.erb
index bd6c586..9d6e1a8 100644
--- a/app/views/tags/new.html.erb
+++ b/app/views/tags/new.html.erb
@@ -1,7 +1,7 @@
 <div class="center-contents">
     <div></div>
     <div id="tag-new">
-        <h2><%= I18n.t('tags.new') %> <span>&#8212; <%= link_to I18n.t('general.back-to-list'), tags_path %></span></h2>
+        <h3><%= I18n.t('tags.new') %> <span>&#8212; <%= link_to I18n.t('general.back-to-list'), tags_path %></span></h3>
 
         <%= render "form", tag: @tag %>
     </div>
-- 
cgit v1.2.3