Closed
Bug 430975
Opened 17 years ago
Closed 17 years ago
app-license.html is missing closing </p>
Categories
(Firefox :: General, defect)
Firefox
General
Tracking
()
RESOLVED
FIXED
People
(Reporter: alqahira, Assigned: alqahira)
References
()
Details
Attachments
(1 file)
|
705 bytes,
patch
|
Gavin
:
review+
damons
:
approval1.9+
|
Details | Diff | Splinter Review |
In bug 368091, I seem to have accidentally landed app-license.html without a closing </p> tag :( As long as license.html remains html, this just breaks validation, but it's wrong nonetheless :(
Gavin, you're the reviewer of record on the browser bits; can you r this goof-fix?
Attachment #317907 -
Flags: review?(gavin.sharp)
Updated•17 years ago
|
Attachment #317907 -
Flags: review?(gavin.sharp) → review+
| Assignee | ||
Comment 1•17 years ago
|
||
Comment on attachment 317907 [details] [diff] [review]
fix
Requesting approval to land this typo-fix. There's no risk, and it makes the Firefox copy of about:license valid HTML again (and will be needed if bug 423104 makes about:license xhtml).
Attachment #317907 -
Flags: approval1.9?
Comment 2•17 years ago
|
||
Comment on attachment 317907 [details] [diff] [review]
fix
a1.9+=damons
Attachment #317907 -
Flags: approval1.9? → approval1.9+
| Assignee | ||
Comment 3•17 years ago
|
||
Checking in browser/base/content/overrides/app-license.html;
/cvsroot/mozilla/browser/base/content/overrides/app-license.html,v <-- app-license.html
new revision: 1.2; previous revision: 1.1
done
Status: NEW → RESOLVED
Closed: 17 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•