From 5a15633a1179f960e1726eebf7f38c7ee2f2b4f8 Mon Sep 17 00:00:00 2001 From: Frédéric Buclin Date: Sat, 28 Dec 2013 20:55:54 +0100 Subject: Bug 952795: Remove all style="..." attributes from HTML elements r/a=justdave --- template/en/default/admin/flag-type/edit.html.tmpl | 25 +++---- template/en/default/admin/params/common.html.tmpl | 4 +- template/en/default/admin/params/general.html.tmpl | 3 +- .../en/default/admin/products/updated.html.tmpl | 2 +- template/en/default/admin/table.html.tmpl | 4 +- template/en/default/admin/users/edit.html.tmpl | 2 +- template/en/default/attachment/diff-file.html.tmpl | 6 +- .../en/default/attachment/diff-header.html.tmpl | 22 ++++-- .../en/default/bug/create/create-guided.html.tmpl | 1 - template/en/default/bug/dependency-tree.html.tmpl | 29 +++----- template/en/default/bug/summarize-time.html.tmpl | 17 ++--- template/en/default/flag/list.html.tmpl | 4 +- template/en/default/global/textarea.html.tmpl | 2 - template/en/default/list/edit-multiple.html.tmpl | 21 +++--- template/en/default/list/server-push.html.tmpl | 2 +- template/en/default/list/table.html.tmpl | 4 +- template/en/default/reports/create-chart.html.tmpl | 48 ++++++------- template/en/default/reports/edit-series.html.tmpl | 4 +- .../en/default/reports/series-common.html.tmpl | 4 +- template/en/default/reports/series.html.tmpl | 44 ++++++------ .../default/search/search-create-series.html.tmpl | 2 +- template/en/default/whine/schedule.html.tmpl | 84 +++++++++------------- 22 files changed, 155 insertions(+), 179 deletions(-) (limited to 'template') diff --git a/template/en/default/admin/flag-type/edit.html.tmpl b/template/en/default/admin/flag-type/edit.html.tmpl index 2e9771384..ec19c0496 100644 --- a/template/en/default/admin/flag-type/edit.html.tmpl +++ b/template/en/default/admin/flag-type/edit.html.tmpl @@ -23,10 +23,7 @@ [% PROCESS global/header.html.tmpl title = title - style = " - table#form th { text-align: right; vertical-align: baseline; white-space: nowrap; } - table#form td { text-align: left; vertical-align: baseline; } - " + style_urls = ['skins/standard/admin.css'] onload="var f = document.forms['flagtype_properties']; selectProduct(f.product, f.component, null, null, '__Any__');" javascript_urls=["js/productform.js"] @@ -49,7 +46,7 @@ [%# Add a hidden button at the top of the form so that the user pressing "return" # really submit the form, as expected. %] - + @@ -90,8 +87,8 @@ [% END %]
-
- Product/Component:
+ 		+ Product/Component:
- - 		- Inclusions:
+ + 		+ Inclusions:
[% PROCESS category_select name="inclusion_to_remove" categories = inclusions %]
- - 		- Exclusions:
+ + 		+ Exclusions:
[% PROCESS category_select name="exclusion_to_remove" categories = exclusions %]
- +
diff --git a/template/en/default/admin/params/common.html.tmpl b/template/en/default/admin/params/common.html.tmpl index 5f6f1bfc0..db3bde777 100644 --- a/template/en/default/admin/params/common.html.tmpl +++ b/template/en/default/admin/params/common.html.tmpl @@ -69,14 +69,14 @@ [% END %] - + - + diff --git a/template/en/default/admin/params/general.html.tmpl b/template/en/default/admin/params/general.html.tmpl index 369e495c7..9de306d1d 100644 --- a/template/en/default/admin/params/general.html.tmpl +++ b/template/en/default/admin/params/general.html.tmpl @@ -49,8 +49,7 @@ _ " green text inside a red box, or class=\"bz_private\" for" _ " dark red on a red background. Anything defined in " _ " skins/standard/global.css or skins/custom/global.css" - _ " will work. To get centered text, use style=\"text-align: " - _ " center;\".", + _ " will work. To get centered text, use class=\"center\".", upgrade_notification => "$terms.Bugzilla can inform you when a new release is available." diff --git a/template/en/default/admin/products/updated.html.tmpl b/template/en/default/admin/products/updated.html.tmpl index 4d2280163..cc0c9f8ea 100644 --- a/template/en/default/admin/products/updated.html.tmpl +++ b/template/en/default/admin/products/updated.html.tmpl @@ -39,7 +39,7 @@

Updated description to:

-

[% product.description FILTER html_light %]

+

[% product.description FILTER html_light %]

[% END %] [% IF changes.isactive.defined %] diff --git a/template/en/default/admin/table.html.tmpl b/template/en/default/admin/table.html.tmpl index b8492dcb2..3981ef4bb 100644 --- a/template/en/default/admin/table.html.tmpl +++ b/template/en/default/admin/table.html.tmpl @@ -67,11 +67,11 @@ [%################### TABLE HEADER ######################%] - +
[% FOREACH c = columns %] [%# Default to align left for headers %] - [% END %] diff --git a/template/en/default/admin/users/edit.html.tmpl b/template/en/default/admin/users/edit.html.tmpl index b75f8400d..e301ed21e 100644 --- a/template/en/default/admin/users/edit.html.tmpl +++ b/template/en/default/admin/users/edit.html.tmpl @@ -52,7 +52,7 @@ [% IF editusers %] - + [% END %] diff --git a/template/en/default/attachment/diff-file.html.tmpl b/template/en/default/attachment/diff-file.html.tmpl index 2c87188ad..1af6b3140 100644 --- a/template/en/default/attachment/diff-file.html.tmpl +++ b/template/en/default/attachment/diff-file.html.tmpl @@ -14,7 +14,11 @@ # considers the first row to determine column widths. If a colspan is found, # it then share the width equally among all columns, which we don't want. #%] -
+ [% c.heading FILTER html %]
||User is a member of these groups
[% collapsed ? '(+)' : '(-)' %] + [%# Hide/show resolved button Swaps text depending on the state of hide_resolved %] - diff --git a/template/en/default/bug/summarize-time.html.tmpl b/template/en/default/bug/summarize-time.html.tmpl index 932a6e3f4..ee654dc38 100644 --- a/template/en/default/bug/summarize-time.html.tmpl +++ b/template/en/default/bug/summarize-time.html.tmpl @@ -36,7 +36,7 @@ [%# parts contains date ranges (from, to). %] [% part = parts.shift %]
-

+

[% IF part.0 or part.1 %] [% part.0 OR "Up" FILTER html %] to [% part.1 OR "now" FILTER html %] [% ELSE %] @@ -52,7 +52,7 @@ [% END %] [% IF detailed %] -

+

Total of [% global.remaining FILTER format("%.2f") %]h remains from original estimate of [% global.estimated FILTER format("%.2f") %]h [% IF global.deadline %] @@ -62,13 +62,13 @@ [% END %] [% IF monthly %] -

Total of [% global.grand_total FILTER format("%.2f") %] hours worked

+

Total of [% global.grand_total FILTER format("%.2f") %] hours worked

[% END %] [% IF null.size > 0 %] [% INCLUDE inactive_report %] -

Total of [% null.size %] inactive [% terms.bugs %]

+

Total of [% null.size %] inactive [% terms.bugs %]

[% END %] [% END %] @@ -255,15 +255,16 @@ [% BLOCK query_form %]
-
+ [% IF warn_swap_dates %] -

The - end date specified occurs before the start date, which doesn't - make sense; the dates below have therefore been swapped.

+

+ The end date specified occurs before the start date, which doesn't + make sense; the dates below have therefore been swapped. +

[% END %]
[% collapsed ? '(+)' : '(-)' %] [% IF lxr_prefix && !file.is_add %] [% file.filename FILTER html %] [% ELSE %] diff --git a/template/en/default/attachment/diff-header.html.tmpl b/template/en/default/attachment/diff-header.html.tmpl index 4261c8423..b8b0d2cc0 100644 --- a/template/en/default/attachment/diff-header.html.tmpl +++ b/template/en/default/attachment/diff-header.html.tmpl @@ -43,7 +43,9 @@ Interdiff of #[% oldid %] and #[% newid %] for [% terms.bug %] #[% bugid %] [% ELSE %] - + @@ -57,13 +59,15 @@ Interdiff of #[% oldid %] and #[% newid %] for [% terms.bug %] #[% bugid %] [% IF headers %] View | Details - | Raw Unified + | Raw Unified | Return to [% "$terms.bug $bugid" FILTER bug_link(bugid) FILTER none %] [% END %] [% IF other_patches.size > 0 %] [% IF headers %] |[%END%] Differences between -
+
) +
+ + + + + +
) [% END %] [% IF warning %] @@ -139,4 +149,4 @@ Interdiff of #[% oldid %] and #[% newid %] for [% terms.bug %] #[% bugid %] [%# Restore Stuff %]
- + diff --git a/template/en/default/bug/create/create-guided.html.tmpl b/template/en/default/bug/create/create-guided.html.tmpl index 181b76e62..7abb8c7dc 100644 --- a/template/en/default/bug/create/create-guided.html.tmpl +++ b/template/en/default/bug/create/create-guided.html.tmpl @@ -62,7 +62,6 @@ function PutDescription() {

diff --git a/template/en/default/bug/dependency-tree.html.tmpl b/template/en/default/bug/dependency-tree.html.tmpl index 369326d48..61994cf28 100644 --- a/template/en/default/bug/dependency-tree.html.tmpl +++ b/template/en/default/bug/dependency-tree.html.tmpl @@ -166,9 +166,8 @@
- + + [% IF maxdepth %] @@ -188,11 +187,9 @@ - + [%# set to one form %] - @@ -201,8 +198,7 @@ - + [%# Minus one form Allow subtracting only when realdepth and maxdepth > 1 %] @@ -211,14 +207,13 @@ : ( maxdepth ? maxdepth - 1 : realdepth - 1 ) %]"> - -
+ [%# Limit entry form: the button cannot do anything when total depth is less than two, so disable it %] + [%# plus one form Disable button if total depth < 2, or if depth set to unlimited %] @@ -243,19 +237,18 @@ [% END %] - = realdepth %]>
-
+ [%# Unlimited button %]
diff --git a/template/en/default/flag/list.html.tmpl b/template/en/default/flag/list.html.tmpl index 4d7eefb48..8acef5631 100644 --- a/template/en/default/flag/list.html.tmpl +++ b/template/en/default/flag/list.html.tmpl @@ -132,9 +132,8 @@ [% IF any_flags_requesteeble %] - [% END %] diff --git a/template/en/default/global/textarea.html.tmpl b/template/en/default/global/textarea.html.tmpl index 216f2ddbf..bc5fd6311 100644 --- a/template/en/default/global/textarea.html.tmpl +++ b/template/en/default/global/textarea.html.tmpl @@ -11,7 +11,6 @@ # id: (optional) The "id"-attribute of the textarea. # name: (optional) The "name"-attribute of the textarea. # accesskey: (optional) The "accesskey"-attribute of the textarea. - # style: (optional) The "style"-attribute of the textarea. # classes: (optional) The "class"-attribute of the textarea. # wrap: (deprecated; optional) The "wrap"-attribute of the textarea. # disabled: (optional) Disable the textarea. @@ -33,7 +32,6 @@
+ [% IF (type.is_active && type.is_requestable && type.is_requesteeble) || (flag && flag.requestee) %] - [% SET grant_list = [] %] [% IF Param('usemenuforusers') %] [% grant_list = type.grant_list %] @@ -158,7 +157,6 @@ classes => ["requestee"] custom_userlist => grant_list %] - [% END %]