Closed Bug 952795 Opened 11 years ago Closed 11 years ago

Remove all style="..." attributes from HTML elements

Categories

(Bugzilla :: User Interface, enhancement)

4.5.1
enhancement
Not set
normal

Tracking

()

RESOLVED FIXED
Bugzilla 5.0

People

(Reporter: LpSolit, Assigned: LpSolit)

References

Details

Attachments

(1 file)

<foo style="..."> should be replaced by CSS rules.
Attached patch patch, v1Splinter Review
style="..." is still in use in email/bugmail.html.tmpl because CSS rules must be included with the bugmails, and it's also in use in search/boolean-charts.html.tmpl to indent rows correctly depending on the AND/OR combinations. As there is no limit in the number of AND/OR you can use, we cannot hardcode the list in CSS files. All other style="..." have been removed.
Assignee: ui → LpSolit
Status: NEW → ASSIGNED
Attachment #8351205 - Flags: review?(justdave)
Comment on attachment 8351205 [details] [diff] [review]
patch, v1

Looks good, I don't see any issues.
Attachment #8351205 - Flags: review?(justdave) → review+
Flags: approval+
Committing to: bzr+ssh://lpsolit%40gmail.com@bzr.mozilla.org/bugzilla/trunk/
modified skins/standard/admin.css
modified skins/standard/dependency-tree.css
modified skins/standard/editusers.css
modified skins/standard/global.css
modified skins/standard/reports.css
modified skins/standard/summarize-time.css
modified template/en/default/admin/table.html.tmpl
modified template/en/default/admin/flag-type/edit.html.tmpl
modified template/en/default/admin/params/common.html.tmpl
modified template/en/default/admin/params/general.html.tmpl
modified template/en/default/admin/products/updated.html.tmpl
modified template/en/default/admin/users/edit.html.tmpl
modified template/en/default/attachment/diff-file.html.tmpl
modified template/en/default/attachment/diff-header.html.tmpl
modified template/en/default/bug/dependency-tree.html.tmpl
modified template/en/default/bug/summarize-time.html.tmpl
modified template/en/default/bug/create/create-guided.html.tmpl
modified template/en/default/flag/list.html.tmpl
modified template/en/default/global/textarea.html.tmpl
modified template/en/default/list/edit-multiple.html.tmpl
modified template/en/default/list/server-push.html.tmpl
modified template/en/default/list/table.html.tmpl
modified template/en/default/reports/create-chart.html.tmpl
modified template/en/default/reports/edit-series.html.tmpl
modified template/en/default/reports/series-common.html.tmpl
modified template/en/default/reports/series.html.tmpl
modified template/en/default/search/search-create-series.html.tmpl
modified template/en/default/whine/schedule.html.tmpl
Committed revision 8841.
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Blocks: 301384
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: