506167 - [ForumUX] Represent "solution proposed" and "request for more info" statuses in thread list

Status: VERIFIED FIXED

(support.mozilla.org :: Forum, task, P3)

Description

[[:Cww]]

Possibly icons but awaiting UI review for possible better ideas from neil.

Comment 1

[Laura Thomson :laura]

Neil, can you please comment on this one?

Comment 2

[Neil Lee [:neilio]]

This states are only visible to contributors and not end-users? I think adding new icons to replace the default "document" icon is probably the best approach - stays consistent with the current usage of "this is answered".

Not sure what the yield sign currently means, but we could use a icon with a ? for "more info required" and a document with a question mark overlay for "solution proposed".

Comment 3

[[:Cww]]

These states would also be visible to end users but really end users shouldn't be using the forum listing view.

Comment 4

[Paul Craciunoiu [:paulc]]

Attachment: v1

Adds the desired functionality.
Unfortunately, we still need the icons. I discussed this with Cheng a while back, and having the functionality helps. We can then add the icons at anytime, when they're ready. Hence, what I did was to copy over the "normal" status icon and look at the title attribute:
---
cp webroot/img/silk/page.png webroot/img/silk/proposed.png
cp webroot/img/silk/page.png webroot/img/silk/request.png
---
With this patch, we won't have to make any code changes when the icons come in, which is pretty cool.

Comment 5

[James Socol [:jsocol, :james]]

Comment on attachment 404477 [details] [diff] [review]
v1

Looks good to me. Hopefully we'll get the new icons soon.

Comment 6

[Paul Craciunoiu [:paulc]]

r53852 (trunk).

Stephend: Currently, the icons look the same as for "normal" status. Hover over the icons to see the "hint" (title, tooltip, whatever-you-call it), it should say "request for more information" or "solution proposed"

I emailed Marius and he will let us know about the icons.

Comment 7

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

Attachment: Screenshot of "request for more information" tooltip

Comment 8

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

Attachment: Screenshot of "proposed solution" tooltip

Comment 9

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

Verified FIXED.

Comment 10

[Paul Craciunoiu [:paulc]]

I'm thinking to wait for the icons before committing this to prod.

Comment 11

[Paul Craciunoiu [:paulc]]

r54615 and r54616 (prod) -- sorry for double commit, I forgot the images first time, even tho I had copied them over.
r54619 (fennec) -- only images added (patch failed on comment-icon.tpl)