596469 - Discover Pane - add-on descriptions allow HTML because we're using its description, rather than its summary

Status: VERIFIED FIXED

(addons.mozilla.org Graveyard :: Discovery Pane, defect)

Description

[Stephen Donner [:stephend] Not actively reading bugmail]

STR:

1. Load https://services.addons.mozilla.org/en-US/firefox/discovery/3.6/Darwin
2. Look at the FoxTab entry

Expected Results:

"New!..."

Actual Results:

<b>New!</b> -- we display HTML because we're using the Summary, rather than the Description.

Comment 1

[Stephen Donner [:stephend] Not actively reading bugmail]

Attachment: Screenshot

Comment 2

[Justin Scott [:fligtar]]

Other way around. Summaries don't allow HTML and are short and what we should be using. Description are long and allow HTML and should not be used. I switched to summary in one spot (collections) but there was another spot that was missed.

We realized this during the push and Jeff might have already fixed it on preview.

Comment 3

[Stephen Donner [:stephend] Not actively reading bugmail]

Jeff landed a fix for this in http://github.com/jbalogh/zamboni/commit/f973ca309be68f68f2dbb721aa2482a656b743db.

Comment 4

[Jeff Balogh (:jbalogh)]

You are a man of strange ceremony, stephend.

Comment 5

[Stephen Donner [:stephend] Not actively reading bugmail]

(In reply to comment #4)
> You are a man of strange ceremony, stephend.

You taught me well.

Verified FIXED; we're using the <summary>, now.