Closed Bug 503505 Opened 16 years ago Closed 16 years ago

Update info should allow more html tags through

Categories

(Toolkit :: Add-ons Manager, enhancement)

Product:
Toolkit
Component:
Add-ons Manager
Type:
enhancement
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED FIXED
Tracking Flags:
Tracking Status
status1.9.2 --- .2-fixed
status1.9.1 --- .9-fixed

People

(Reporter: jwkbugzilla, Assigned: mossop)

References

Details

(Keywords: dev-doc-complete, verified1.9.2)

Attachments

(2 files, 1 obsolete file)

screenshot
132.34 KB, image/png
dveditz
: review+
Details
patch rev 2
2.41 KB, patch
robert.strong.bugs
: review+
beltzner
: approval1.9.2.2+
beltzner
: approval1.9.1.9+
Details | Diff | Splinter Review
I tried to show a fairly simple update info for Adblock Plus development builds - list of change descriptions with the corresponding dates. First attempt was with dictionary lists - dl/dt/dd. Result was a blank page. After consulting updateinfo.xsl I tried a plain list with two paragraphs per list item - no success again. The br tag isn't allowed either so displaying more than one line per list item is out of question. In the end I had to give up on lists altogether, would be nice to have a little more flexibility here however.
Assignee: nobody → dtownsend
Status: NEW → ASSIGNED
Summary: Update info should allow dl/dt/dd and br tags → Update info should allow more html tags through
Blocks: 540968
Attached patch patch rev 1 (obsolete) — — Splinter Review
This is a first WIP. It removes a lot of the context based restrictions that currently exist and allow html to be formed pretty much anyway we like. Any elements not included in the list for specific actions are stripped but their contents are still included in the display. A set of formatting, block formatting and list formatting tags are allowed to pass through (with all attributes stripped). <br> and <hr> elements are passed through but their contents are ignored. <head>, <style> and <script> are completely stripped because they normally contain various text contents that aren't meant to be human readable.
Comparing with http://addons.songbirdnest.com/formatting-help.html - xhtml:u is missing as allowed inline element. Also, it would be nice to support xhtml:a but that's not just a matter of templates of course, link clicks need to be handled. As to AMO, they allow xhtml:abbr/xhtml:acronym with title attribute.
<u> I can add to the list. I intentionally left <abbr> and <acronym> out since I think allowing tooltips here would probably violate our need to avoid UI spoofing. We cannot support anchors right now for technical reasonsthat will hopefully be solved in a future release of Firefox.
Attached image screenshot —
Dan, I'd like your quick approval on this. It is a screenshot showing the new range of formatting available to add-ons in the update information and I'd just like to confirm that you see no issues with what we're allowing them to do with this change.
Attachment #424339 - Flags: review?(dveditz)
Comment on attachment 424339 [details] screenshot r=dveditz
Attachment #424339 - Flags: review?(dveditz) → review+
Attached patch patch rev 2 — — Splinter Review
This is the patch that implements the changes shown.
Attachment #422650 - Attachment is obsolete: true
Attachment #429022 - Flags: review?(robert.bugzilla)
The test extension at https://people.mozilla.com/~dtownsend/testcases/bug503505/test.xpi was the one used to generate the screenshot.
Attachment #429022 - Flags: review?(robert.bugzilla) → review+
Landed: http://hg.mozilla.org/mozilla-central/rev/fe2240e4d3d3
Status: ASSIGNED → RESOLVED
Closed: 16 years ago
Resolution: --- → FIXED
Whiteboard: [needs baking]
Keywords: dev-doc-needed
Comment on attachment 429022 [details] [diff] [review] patch rev 2 Backporting this to the branches would really help add-on developers and the AMO team.
Attachment #429022 - Flags: approval1.9.2.2?
Attachment #429022 - Flags: approval1.9.1.9?
Whiteboard: [needs baking]
Comment on attachment 429022 [details] [diff] [review] patch rev 2 a=beltzner for both branches
Attachment #429022 - Flags: approval1.9.2.2?
Attachment #429022 - Flags: approval1.9.2.2+
Attachment #429022 - Flags: approval1.9.1.9?
Attachment #429022 - Flags: approval1.9.1.9+
verified with: Mozilla/5.0 (Macintosh; U; Intel Mac OS X 10.6; en-US; rv:1.9.2.3pre) Gecko/20100319 Namoroka/3.6.3pre
Keywords: verified1.9.2
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: