527930 - Implement the GS spec

Status: VERIFIED FIXED

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

Description

[Wil Clouser [:clouserw]]

I've got a JS file to host locally, other than that we just need to store the company name and potentially product name.

Comment 1

[Wil Clouser [:clouserw]]

Attachment: Screenshot - on the page

I'm boarding the plane in 6 minutes so here are some screenshots for feedback.  First:

Looking for feedback on wording.  I think we could fix bug 499482 at the same time, so assume we remove the "support" section in the middle of the page if we have this box in the top right.  I can make the box as wide as the others if you want - using the prose class squished it down.

Comment 2

[Wil Clouser [:clouserw]]

Attachment: Screenshot: Developer tools

This is editing the properties of an add-on.  These are the only two fields we need.  Basically looking for feedback on wording again.  This will show up for all locales but the screenshot above (on the page) will only show up if the user is using en-US.

Comment 3

[Wil Clouser [:clouserw]]

Attachment: Screenshot: overlay

Here is a screenshot of the actual GS widget once activated on the page.  This is all defaults.  There isn't good documentation available for this widget but we can do whatever we want via CSS.  There is a close button on the top left, but it didn't load because the wireless here sucks.

Comment 4

[Nick Nguyen [:osunick]]

Is there any way to embed a widget inline?  I'm worried the single link might not be discoverable.

Comment 5

[Wil Clouser [:clouserw]]

(In reply to comment #4)
> Is there any way to embed a widget inline?  I'm worried the single link might
> not be discoverable.

It didn't fit in that column very well when I tried it.

Comment 6

[Justin Scott [:fligtar]]

Box:
Yeah, this should definitely be the same width as the others. Perhaps for copy, make it more obvious that this is help for the specific add-on rather than help with add-ons or Firefox in general. Let's also promote Get Satisfaction first if it's available for the add-on.

Need help with this add-on?
* Ask others on _Get_Satisfaction_
* Visit the _support_website_
* _E-mail_ your question

On the Dev tools:
I think this is good. Perhaps the Product explanation could use an example like in the first one to help people figure out what to put in the box.

Widget overlay:
This looks good

Comment 7

[Wil Clouser [:clouserw]]

Attachment: patch

Thanks fligtar.  I tweaked the wording a bit

Comment 8

[Neil Lee [:neilio]]

I'm not sure if having the support stuff in the sidebar is the best placement - it will probably get lost there and not noticed. 

If we're locked on having the support box in the sidebar, I'd move it up to be the very first box. Getting support is more important to most users than information about the developer or related add-ons.

I think the best location is to have a call-out box that appears above the details table - again, knowing how to find support is probably a bigger priority for an add-on's users than that add-on's minutiae.

If this is something we'd like to pursue I can do a mockup - let me know.

Comment 9

[Wil Clouser [:clouserw]]

Moving it to the sidebar is (In reply to comment #8)
> I'm not sure if having the support stuff in the sidebar is the best placement -
> it will probably get lost there and not noticed. 
> 
> If we're locked on having the support box in the sidebar, I'd move it up to be
> the very first box. Getting support is more important to most users than
> information about the developer or related add-ons.
> 
> I think the best location is to have a call-out box that appears above the
> details table - again, knowing how to find support is probably a bigger
> priority for an add-on's users than that add-on's minutiae.
> 
> If this is something we'd like to pursue I can do a mockup - let me know.

I think moving it to the sidebar is a lot better than where it is now, and I'd rather have it on the side than in the middle.  That area is already crowded.  If we want to emphasize it there are other ways like tweaking the background color or giving it a border or fancy-ish background image but let's try this and see how it goes.

Comment 10

[Fred Wenzel [:wenzel]]

Comment on attachment 412937 [details] [diff] [review]
patch

It works well, almost completely, but when I click on the link in the sidebar, the dark overlay over the page only spans the content part of the page, not the entire thing (hard to explain, I'll add a screenshot). That could be an un-closed <div>  somewhere or something. If you fix this, it's an r+.

Comment 11

[Fred Wenzel [:wenzel]]

Attachment: Bug Screenshot: Partial Overlay

That's what I was referring to: the dark gray does not span all the way across the page.

Comment 12

[Wil Clouser [:clouserw]]

Attachment: incremental fix

Comment 13

[Fred Wenzel [:wenzel]]

Comment on attachment 413355 [details] [diff] [review]
incremental fix

Faaaantastic.

Comment 14

[Wil Clouser [:clouserw]]

r56437

Comment 15

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

Attachment: Post-fix screenshot

Comment 16

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

Verified FIXED on https://preview.addons.mozilla.org/en-US/firefox/addon/9331.