602997 - Add listing page for all opinions in a theme

Status: VERIFIED FIXED

(Input :: General, enhancement, P2)

Description

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

The "[num] similar messages" text isn't linked on http://input.stage.mozilla.com/en-US/themes/

(Prod neither.)

Comment 1

[Dave Dash [:davedash, :dd] (assign all bugs to mbrandt)]

These aren't supposed to be linked.

Comment 2

[Aakash Desai [:aakashd]]

Yeah, they are supposed to be linked. 

http://people.mozilla.com/~chowse/drop/mozilla.com/input/style/themes.html

Is this possible to get done by Tuesday so QA says Wednesday and Thursday to test it, Dave?

Comment 3

[Dave Dash [:davedash, :dd] (assign all bugs to mbrandt)]

This should be easy <1 day, if even that.

Comment 4

[Fred Wenzel [:wenzel]]

Where do you want to link them to? From the chowse designs, it looks like these just link to the themes page, just like the theme's topic line right above. Not sure if that's helpful. I am fine with WONTFIXing or fixing this.

Aakash, will you swing the sword of product decision on these helpless letters, then push this bug back to me?

Comment 5

[Aakash Desai [:aakashd]]

They should link to the same pages that are linked to from the clusters on sites. The ones I'm talking about can be found by:

1. Going to http://input.mozilla.com/en-US/sites
2. Click on "xx similar" link next to a clustered opinion within a domain.


The reason for this is to stay consistent in functionality between sites and themes as well as offer us an idea of the messages that constituted this clustering.

Comment 6

[Fred Wenzel [:wenzel]]

Ah, looks like we don't have a per-theme listing page yet. That's unfortunate.

Comment 7

[Fred Wenzel [:wenzel]]

Morphing this into a bug to add a listing page per theme. Will then need to link to it from the "Similar messages" text.

Also pushing this to 2.1 as it is a new feature (though the UX design did already anticipate this).

Comment 8

[Michael Kurze [:michaelk]]

Fixed. Also made small changes to the themes listing so as to better match sites.

https://github.com/fwenzel/reporter/commit/5a1a6795a05cf6b7176155f281036a03efed4048

Comment 9

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

Verified FIXED on http://input.stage.mozilla.com/en-US/themes/89904.

Comment 10

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

Attachment: Post-fix screenshot