Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Unified Diff: sitescripts/reports/template/submitSuccess.html

Issue 29584689: Issue 5890 - Align the font of the issue reporter result page with the font of the extension UI (Closed)
Patch Set: Created Oct. 20, 2017, 3:44 p.m.
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: sitescripts/reports/template/submitSuccess.html
===================================================================
--- a/sitescripts/reports/template/submitSuccess.html
+++ b/sitescripts/reports/template/submitSuccess.html
@@ -13,16 +13,23 @@
-
- You should have received a copy of the GNU General Public License
- along with Adblock Plus. If not, see <http://www.gnu.org/licenses/>.
-->
<html xmlns="http://www.w3.org/1999/xhtml">
<head>
<title>Report submission</title>
+ <style type="text/css">
+body {
Wladimir Palant 2017/10/21 18:14:42 Nit: this bracket belongs on the next line.
Vasily Kuznetsov 2017/10/22 09:59:31 Done.
+ font-family: Segoe UI, Arial, sans-serif;
+ font-size: 1.25rem;
Vasily Kuznetsov 2017/10/20 15:53:41 I used 1.25rem instead of 20px as in the ticket si
Wladimir Palant 2017/10/21 18:14:42 The trouble is that I likely need to change that i
Vasily Kuznetsov 2017/10/22 09:59:31 Done.
+ color: #494949;
Wladimir Palant 2017/10/21 18:14:42 You cannot define text color without setting backg
Vasily Kuznetsov 2017/10/22 09:59:31 Done.
+}
+ </style>
</head>
<body>
{% if knownIssues -%}
<p><strong>%KNOWNISSUE%</strong>
{%- for issue in knownIssues -%}
<br /><a target="_top" href="{{issue}}">{{issue}}</a>
{%- endfor -%}
</p>
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld