From 42099f24eea1a3858d980e707a3333854a14e0e9 Mon Sep 17 00:00:00 2001
From: Jesse Plamondon-Willard <Pathoschild@users.noreply.github.com>
Date: Tue, 5 Jul 2022 22:36:02 -0400
Subject: improve log parser's 'suggested fixes' styles

---
 docs/release-notes.md                            |  4 ++-
 src/SMAPI.Web/Views/LogParser/Index.cshtml       |  4 +--
 src/SMAPI.Web/wwwroot/Content/css/log-parser.css | 34 +++++++++++++++++++++---
 3 files changed, 35 insertions(+), 7 deletions(-)

diff --git a/docs/release-notes.md b/docs/release-notes.md
index 8361a780..a94bb295 100644
--- a/docs/release-notes.md
+++ b/docs/release-notes.md
@@ -18,7 +18,9 @@
   * The [FluentHttpClient package](https://github.com/Pathoschild/FluentHttpClient#readme) is now loaded by SMAPI.
 
 * For the web UI:
-  * The log parser now detects a missing or outdated Error Handler mod for its 'suggested fixes' section.
+  * Improved the log parser's 'suggested fixes' section:
+    * added warning if Error Handler is missing or outdated;
+    * improved visual styles.
   * Updated the JSON validator/schema for Content Patcher 1.27.0.
   * Fixed the mod count in the log parser metadata.
 
diff --git a/src/SMAPI.Web/Views/LogParser/Index.cshtml b/src/SMAPI.Web/Views/LogParser/Index.cshtml
index 8bf5d86d..b824b768 100644
--- a/src/SMAPI.Web/Views/LogParser/Index.cshtml
+++ b/src/SMAPI.Web/Views/LogParser/Index.cshtml
@@ -39,7 +39,7 @@
         <meta name="robots" content="noindex" />
     }
     <link rel="stylesheet" href="~/Content/css/file-upload.css" />
-    <link rel="stylesheet" href="~/Content/css/log-parser.css?r=20220409" />
+    <link rel="stylesheet" href="~/Content/css/log-parser.css" />
     <link rel="stylesheet" href="https://cdn.jsdelivr.net/npm/tabbyjs@12.0.3/dist/css/tabby-ui-vertical.min.css" />
 
     <script src="https://cdn.jsdelivr.net/npm/tabbyjs@12.0.3" crossorigin="anonymous"></script>
@@ -248,7 +248,7 @@ else if (log?.IsValid == true)
             <ul id="fix-list">
                 @if (errorHandler is null)
                 {
-                    <li>You don't have the <strong>Error Handler</strong> mod installed. This automatically prevents many game or mod errors. You can <a href="https://stardewvalleywiki.com/Modding:Player_Guide#Install_SMAPI">reinstall SMAPI</a> to re-add it.</li>
+                    <li class="important">You don't have the <strong>Error Handler</strong> mod installed. This automatically prevents many game or mod errors. You can <a href="https://stardewvalleywiki.com/Modding:Player_Guide#Install_SMAPI">reinstall SMAPI</a> to re-add it.</li>
                 }
                 @if (hasOlderErrorHandler)
                 {
diff --git a/src/SMAPI.Web/wwwroot/Content/css/log-parser.css b/src/SMAPI.Web/wwwroot/Content/css/log-parser.css
index 1d457e35..f136a96f 100644
--- a/src/SMAPI.Web/wwwroot/Content/css/log-parser.css
+++ b/src/SMAPI.Web/wwwroot/Content/css/log-parser.css
@@ -53,6 +53,36 @@ table caption {
     opacity: 0.3;
 }
 
+/*********
+** Suggested fixes
+*********/
+#fix-list {
+    padding-left: 1em;
+    margin-bottom: 2em;
+}
+
+#fix-list li {
+    padding: 0.5em;
+    background: #FFC;
+    border: 1px solid #880;
+    border-radius: 5px;
+    list-style-type: none;
+}
+
+#fix-list li:not(:last-child) {
+     margin-bottom: 0.5em;
+}
+
+#fix-list li.important {
+    background: #FCC;
+    border-color: #800;
+}
+
+#fix-list li::before {
+    content: "⚠ ";
+}
+
+
 /*********
 ** Log metadata & filters
 *********/
@@ -84,10 +114,6 @@ table caption {
     min-height: 1.3em;
 }
 
-#fix-list {
-    margin-bottom: 2em;
-}
-
 #updates {
     min-width: 10em;
 }
-- 
cgit