546830 - [Details Page] Implement Report Abuse HTML/CSS/JS

Status: VERIFIED FIXED

(addons.mozilla.org Graveyard :: Public Pages, defect, P4)

Description

[Justin Scott [:fligtar]]

Please implement the report abuse HTML, CSS, and JS as shown in these mocks:

(before click -- scroll to bottom) http://people.mozilla.com/~fligtar/zamboni/designs/2010-02-15/Addon-100215.png
(after click) http://people.mozilla.com/~fligtar/zamboni/designs/2010-02-15/Addon-AbuseBox-100215.png

Comment 1

[Craig Cook (:craigcook)]

Attachment: Report abuse form; markup, css, images

The attached zip includes:

* Report Abuse form HTML with some example JavaScript to demo the interaction.
* The latest zamboni.css, up to date with your changes as of r64096.
* An updated icons sprite at /media/img/amo2009/icons/icons.png (added the alert icon to the collection, all the existing icons are unchanged).

Online demo is at http://mozilla.focalcurve.com/zamboni/reportabuse.php

Comment 2

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

I didn't see a 5.8 bug that covers the implementation on the site; is this going to land in 5.8?

Comment 3

[Wil Clouser [:clouserw]]

No, this was a design bug

Comment 4

[Wil Clouser [:clouserw]]

(In reply to comment #3)
> No, this was a design bug

Maybe that was bad wording.  This was a front-end bug that Craig did, and back end implementation is upcoming.  :-P  I'll move it to 5.9 to keep track.

Comment 8

[Wil Clouser [:clouserw]]

- This form should show up for everyone
- If the user is logged in, it's as in the mockup.  If the user is not, we show a captcha (that library exists already, see the registration page)
- the backend for this can just email [e-mail] and log.debug() something useful.

Comment 9

[Justin Scott [:fligtar]]

The email should be something like:

Subject: Abuse Report for Password Exporter

(An anonymous user|Justin Scott ([e-mail])) reported abuse for Password Exporter (https://addons.mozilla.org/addon/2848):

[abuse text here]

Comment 10

[Andy McKay]

Is this meant to be on personas as well?

Comment 11

[Wil Clouser [:clouserw]]

Sure

Comment 12

[Jeff Balogh (:jbalogh)]

Report Abuse of what?  There's no explanation of what the whistleblower is supposed to be reporting.

Comment 13

[Andy McKay]

At Jeff's request, added in a global settings flag to toggle this on or off.

https://github.com/jbalogh/zamboni/commit/edb1ff2297b56216ccf1aff62ada2d4d9c3de3fa

Comment 14

[krupa raj[:krupa]]

Checked for the following-

* Check for report abuse on add-on detail page for an a) extension b)persona c)search-tool
* Check the icon
* Report as an anonymous user
* Recaptcha field
* Cancel button functionality
* JS disabled(bug 611894)
* Email content
* /abuse page layout
* Error messages for required fields
* Breadcrumb 
* Reporting abuse in non en-US locales
* JS errors (bug 612824)

Marking this verified