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 942ee9fc89769d2f10ac49392571dd90a650b405..79d18b8a270bd08b13049ec432b21a9863333882 100644 (file)
--- 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 973f7b2ed3a94170791ca634c80e17b3e137d2ee..a4289a4bcf84963002d0ad766c4ae8e6b52aff06 100644 (file)
--- 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 bf6dddc4812bb19390b6da1abcc0462927959747..3fd0c4b98169bb8f1598885b1d46c98f37abd441 100644 (file)
--- 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 689689b39e577f2b8239b6b31975f9c8c34e9dd0..98e1b079ab5440ca3697aee82c053d2d7206702b 100644 (file)
--- 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 ada3e8c6ebedd3a0ffcbc4de0148c52d5ac5e88c..fee1de7e5039bfae2bd9bd170555fe009e0111bd 100644 (file)
--- 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 e01a90a1b1b5fba915eed97f41070e26eade9de4..af1d3018b5430e5e90b1417b201dfdc58a2f44cd 100644 (file)
--- 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 ed761d6896a6de67b814c15c8cbf35b0aa1fd555..5484a0595584bde65ddff5f7a19c59a0a6b1c2a0 100644 (file)
--- 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 ed3ff0f8dbbf69447927b7ad3fbe1b5bf6ddc887..e84ef4083d06544ba1d3b99431ceaecdef2c6a22 100644 (file)
--- 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 938ba809869842614eb9750089efcc3063085c7b..5bc2c1c93f440d4c864a38880bc50d0127b69ae1 100644 (file)
--- 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 {