From: Kohei Yoshino <kohei.yoshino@gmail.com>
Date: Thu, 13 Jun 2019 15:49:31 +0000 (-0400)
Subject: Bug 1558494 - Fix some more issues in the post-Sandstone skin
X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=639f7a89a52591eade6f58ad185fabdcd605f896;p=thirdparty%2Fbugzilla.git

Bug 1558494 - Fix some more issues in the post-Sandstone skin
---

diff --git a/extensions/BMO/template/en/default/bug/create/create-client-bounty.html.tmpl b/extensions/BMO/template/en/default/bug/create/create-client-bounty.html.tmpl
index 942ee9fc8..79d18b8a2 100644
--- a/extensions/BMO/template/en/default/bug/create/create-client-bounty.html.tmpl
+++ b/extensions/BMO/template/en/default/bug/create/create-client-bounty.html.tmpl
@@ -37,7 +37,6 @@
 #bugzilla-body {
     margin-left: auto;
     margin-right: auto;
-    background-color: white;
     max-width: 1024px;
 }
 
diff --git a/extensions/BugModal/template/en/default/bug_modal/edit.html.tmpl b/extensions/BugModal/template/en/default/bug_modal/edit.html.tmpl
index 973f7b2ed..a4289a4bc 100644
--- a/extensions/BugModal/template/en/default/bug_modal/edit.html.tmpl
+++ b/extensions/BugModal/template/en/default/bug_modal/edit.html.tmpl
@@ -178,7 +178,9 @@
         view_only = 1
     %]
       <div id="field-value-bug_id">
-        [% terms.Bug _ " " _ bug.id FILTER none %]
+        <a href="[% basepath _ "show_bug.cgi?id=" _ bug.id FILTER html %]">
+          [%~ terms.Bug _ " " _ bug.id FILTER none ~%]
+        </a>
         [% IF bug.alias %]
           <span class="edit-hide">
             ([% bug.alias FILTER html %])
diff --git a/extensions/BugModal/web/bug_modal.css b/extensions/BugModal/web/bug_modal.css
index bf6dddc48..3fd0c4b98 100644
--- a/extensions/BugModal/web/bug_modal.css
+++ b/extensions/BugModal/web/bug_modal.css
@@ -242,6 +242,10 @@ input[type="number"] {
   font-size: var(--font-size-large);
 }
 
+#field-value-bug_id a {
+  color: inherit;
+}
+
 #field-value-short_desc {
   margin: 0;
   font-size: var(--font-size-h3);
@@ -311,15 +315,23 @@ input[type="number"] {
 
 #user-story {
   margin: 8px;
+  width: calc(100% - 150px);
   min-height: 1em;
-  font-size: inherit;
+  font-size: var(--font-size-large);
+  line-height: var(--line-height-comfortable);
   font-family: inherit;
+  white-space: pre-wrap;
 }
 
 #user-story-actions {
   float: right;
 }
 
+#cf_user_story {
+  width: calc(100% - 150px);
+  font-size: var(--font-size-large);
+}
+
 #new-comment-notice {
   margin-bottom: 50px;
   padding: 20px 8px;
diff --git a/extensions/MyDashboard/web/styles/mydashboard.css b/extensions/MyDashboard/web/styles/mydashboard.css
index 689689b39..98e1b079a 100644
--- a/extensions/MyDashboard/web/styles/mydashboard.css
+++ b/extensions/MyDashboard/web/styles/mydashboard.css
@@ -27,6 +27,11 @@
   font-size: var(--font-size-large) !important;
 }
 
+.yui3-datatable-col-short_desc a:hover,
+.yui3-datatable-col-short_desc a:focus {
+  color: var(--hovered-link-text-color) !important;
+}
+
 #mydashboard .requests {
   margin-bottom: 2em;
 }
diff --git a/extensions/PhabBugz/web/style/phabricator.css b/extensions/PhabBugz/web/style/phabricator.css
index ada3e8c6e..fee1de7e5 100644
--- a/extensions/PhabBugz/web/style/phabricator.css
+++ b/extensions/PhabBugz/web/style/phabricator.css
@@ -59,8 +59,6 @@ span[class^="revision-status-box-"] {
   border: none;
   border-radius: 3px;
   padding: 3px 9px;
-  font-size: var(--font-size-x-large);
-  font-weight: normal;
   white-space: nowrap;
 }
 
diff --git a/skins/standard/attachment.css b/skins/standard/attachment.css
index e01a90a1b..af1d3018b 100644
--- a/skins/standard/attachment.css
+++ b/skins/standard/attachment.css
@@ -245,7 +245,11 @@ div#update_container {
 #viewDiffFrame,
 #viewFrame {
   margin-bottom: 10px;
-  border: 4px solid var(--primary-region-border-color);
+  border: 1px solid var(--control-border-color);
+}
+
+iframe#viewDiffFrame,
+iframe#viewFrame {
   background-color: rgb(255, 255, 255); /* Because the text color in <iframe> cannot be changed */
 }
 
@@ -254,7 +258,12 @@ div#update_container {
   white-space: nowrap;
 }
 
-#editFrame {
+#editFrame,
+textarea#viewDiffFrame,
+textarea#viewFrame {
+  padding: 12px;
+  color: var(--primary-text-color);
+  background-color: var(--primary-text-background-color);
   font-size-adjust: .5;
   font-family: var(--font-family-monospace);
 }
diff --git a/skins/standard/buglist.css b/skins/standard/buglist.css
index ed761d689..5484a0595 100644
--- a/skins/standard/buglist.css
+++ b/skins/standard/buglist.css
@@ -192,7 +192,7 @@ td.bz_total {
 .bz_short_desc_column a:focus,
 .bz_short_short_desc_column a:hover,
 .bz_short_short_desc_column a:focus {
-  color: var(--secondary-label-color);
+  color: var(--hovered-link-text-color);
 }
 
 #request_form #filtering th {
diff --git a/skins/standard/dependency-tree.css b/skins/standard/dependency-tree.css
index ed3ff0f8d..e84ef4083 100644
--- a/skins/standard/dependency-tree.css
+++ b/skins/standard/dependency-tree.css
@@ -8,7 +8,7 @@
 [role="button"] {
   outline: 0;
   border: 0;
-  padding: 0;
+  padding: 0 !important;
   color: inherit;
   background: none transparent !important;
   box-shadow: none !important;
diff --git a/skins/standard/global.css b/skins/standard/global.css
index 938ba8098..5bc2c1c93 100644
--- a/skins/standard/global.css
+++ b/skins/standard/global.css
@@ -2344,6 +2344,8 @@ div.bz_comment_text pre {
   padding: 12px;
   color: var(--primary-text-color);
   background-color: var(--primary-text-background-color);
+  font-size: var(--font-size-x-large);
+  line-height: var(--line-height-comfortable);
 }
 
 pre#comment_preview_text {