Port user review pages to zamboni

RESOLVED FIXED in 5.11.7

Status

addons.mozilla.org Graveyard
Public Pages
P2
normal
RESOLVED FIXED
8 years ago
2 years ago

People

(Reporter: krupa, Assigned: jbalogh)

Tracking

unspecified
5.11.7
Dependency tree / graph

Details

(Whiteboard: [loadtest?], URL)

Attachments

(1 attachment)

(Reporter)

Description

8 years ago
https://preview.addons.mozilla.org/en-US/firefox/reviews/display/<addonID>
should be fully ported to zamboni if we are planning to push add-on details page live in 5.10.
(Assignee)

Updated

8 years ago
Assignee: jbalogh → nobody
Why can't we use the existing remora pages?
Target Milestone: 5.10 → 5.11
(Reporter)

Comment 2

8 years ago
There is already a zamboni page for reviews which is incomplete.
(https://preview.addons.mozilla.org/z/en-US/firefox/addon/1865/reviews/)

I filed a bug for tracking as we plan to push the add-on detail page live in 5.10.

I also think we can fix a few review bugs in the process(http://tinyurl.com/yc3p239)
(Assignee)

Comment 3

8 years ago
(In reply to comment #1)
> Why can't we use the existing remora pages?

We can, we'll just have to temporarily hijack Review.get_url_path.  The buttons and maybe some other things will be inconsistent, but oh well.

(In reply to comment #2)
> I also think we can fix a few review bugs in the
> process(http://tinyurl.com/yc3p239)

Holy scope creep!
Assignee: nobody → jbalogh
Priority: -- → P3
(Assignee)

Updated

8 years ago
Duplicate of this bug: 565497
Blocks: 386625
(Assignee)

Updated

8 years ago
Blocks: 384798
Target Milestone: 5.11 → 5.12
(Assignee)

Updated

8 years ago
Depends on: 568127
Target Milestone: 5.12 → 5.11.1
-> P2 for this bug, not for the scope creep
Priority: P3 → P2
(Assignee)

Updated

8 years ago
Blocks: 522605
(Assignee)

Updated

8 years ago
Blocks: 539669
(Assignee)

Updated

8 years ago
Blocks: 556463
(Assignee)

Updated

8 years ago
Blocks: 559102
(Assignee)

Updated

8 years ago
Blocks: 567064
(Assignee)

Comment 6

8 years ago
Here's what could get rolled into this bug:

# Old bugs
spam filtering (bug 537105)
url for each review (bug 384798)
link to developer review from front page (bug 453128)
show the version the review was for (bug 386625)
rss feed (bug 568127, 480002, 456418)


# Other
show all developer replies in one page
show thread for developer replies?
separate versions from reviews -- necessary for personas


# UI improvements
less emphasis on reporting (bug 508215)
different color/decoration for developer comments

Chowse, do you have any ideas for the UI improvements?  Attachment 392450 [details] shows bug 508215, and I have no idea what we do for dev comments right now.  But we should probably call those out since they're answering user questions.
(In reply to comment #6)
For reporting:

- Replace the <select> with a link that exposes a drop-down. For an example, see
  the 'Add to Collection' in this mock-up:
  http://people.mozilla.com/~chowse/drop/amo/electric-bandwagon/v1/Interactions.png

- Use a smaller typeface.

- Place it (along with the reply/delete actions) after the message.

- (Optionally) Hide or fade the actions until the user hovers over the message.


Other random thoughts:

- Developer responses could be highlighted with color or visual markers
  (icons, labels, etc).

- We need a better visual separation of replies (dotted line, boxes, etc).

- Can replies be nested? If not, do we support blockquoting or backlinks?

- If we can't show an individual review on a page, we could use a 'brighten
  and lighten' effect when the user visits the page through a hash (see
  http://developer.yahoo.com/ypatterns/richinteraction/transition/brighten.html)

Let me see if I can turn these into quick mocks.
(Assignee)

Comment 8

8 years ago
(In reply to comment #7)
> - We need a better visual separation of replies (dotted line, boxes, etc).

Yes!

> - Can replies be nested? If not, do we support blockquoting or backlinks?

Developer replies can be nested, but that's the only method we expose to nest reviews.  We don't have blockquoting or backlinking either, though I've seen reviews trying to talk to each other.  Unlimited nesting is way more out of scope than the other things I'm already including.

> - If we can't show an individual review on a page, we could use a 'brighten
>   and lighten' effect when the user visits the page through a hash

We'll be able to do both.

Thanks chowse.
Created attachment 449642 [details]
Mock-up of Review Page

Here's a mock-up of how the Review page could look w/ the proposed changes, along with a few other random ideas (review sorting, a star rating breakdown, etc.).
Awesome!
To be consistent with our current duplication of data the breadcrumbs should include the <h2> title on the page too.  Otherwise it does look awesome.  I don't think we currently allow people to reply to developers though (or, more accurately, developers are given one reply to any comment and that's it), so the reply box should be for another review.
(Assignee)

Updated

8 years ago
Target Milestone: 5.11.1 → 5.11.2
(In reply to comment #11)
> I don't think we currently allow people to reply to developers though

Sorry, poor documentation on my part. The 'Report' options is available to anyone, the 'Reply' option for authors (and admins?), and the 'Delete' option for admins or the review writer. If the person viewing the page doesn't have permissions, that element should be removed.

> (or, more
> accurately, developers are given one reply to any comment and that's it)

Is that one reply per review, or one reply for ALL reviews (like a developer's bulletin board)?


> so the reply box should be for another review.

Yes. I included it there to show the relationship to the review, but there should be no need for a second reply (but maybe a way to edit the existing reply?)
Target Milestone: 5.11.2 → 5.11.1
(Assignee)

Updated

8 years ago
Target Milestone: 5.11.1 → 5.11.2
(Assignee)

Updated

8 years ago
Blocks: 570617
(Assignee)

Comment 13

8 years ago
chowse: do you want me to freestyle on "See <n> previous reviews from this user"? http://grab.by/4SU1
Latest review mock-ups:

  Complete Version:
  http://people.mozilla.com/~chowse/drop/amo/reviews/Reviews.png

  Simplified Version (no avatars, sorting, all vs.latest version):
  http://people.mozilla.com/~chowse/drop/amo/reviews/Reviews-Simpler.png

  Masters (Omnigraffle):
  http://people.mozilla.com/~chowse/drop/amo/reviews/amo-reviews.graffle.zip

And here's the styling for some of the new elements:

.review {
   background: #fff;
   border: 1px solid #c8e8f2;
   padding: 1em;
   border-radius: .5em;
}

.review .title {
   font-size: 16px;
   font-weight: bold;
}

.review .text-below-title {
   font-size: 11px;
}

.review .footer-text {
   font-size: 11px;
   font-style: italic;
}

.review .footer-links {
   color: #999;
   font-style: normal;
}

.review:hover .footer-links {
   color: /* typical link color */;
}

.review.author-reply {
   background: #f2f9ee;
   border-color: #c6dbb5;
}

.review.author-reply-form {
   background: #c8e8f2;
}

.review.older-version {
   background: #f0f0f0;
   border-color: #e0e0e0;
}

.sidebar .title {
   font-size: 14px;
}

.sidebar .text {
   font-size: 11px;
}

.sidebar .bars {
   background: #93bfcd;
}

.sidebar .middle-bar {
   border-left: 1px solid #999;
}

.sidebar .separators {
   border-bottom: 1px dotted #93bfcd;
}

Everything else should follow the existing Zamboni styling. Feel free to look at the masters if I overlook anything.
(Assignee)

Updated

8 years ago
Blocks: 571917
(Assignee)

Updated

8 years ago
Depends on: 571923
Target Milestone: 5.11.2 → 5.11.3
(Assignee)

Comment 15

8 years ago
The first (non-interactive) part of this landed in http://github.com/jbalogh/zamboni/commit/06939f6.
Target Milestone: 5.11.3 → 5.11.4
(Assignee)

Updated

8 years ago
Depends on: 576648
(Assignee)

Updated

8 years ago
Depends on: 577332
(Assignee)

Updated

8 years ago
Depends on: 577379
(Assignee)

Updated

8 years ago
Depends on: 577381
(Assignee)

Updated

8 years ago
Target Milestone: 5.11.4 → 5.12
(Assignee)

Updated

8 years ago
Blocks: 582462
(Assignee)

Comment 16

8 years ago
All dependent bugs are in 5.11.7, I'm calling this fixed.
Status: NEW → RESOLVED
Last Resolved: 8 years ago
Resolution: --- → FIXED
Whiteboard: [loadtest?]
Target Milestone: 5.12 → 5.11.7
Depends on: 582881
Depends on: 582883
No longer depends on: 582881
Depends on: 582881
Product: addons.mozilla.org → addons.mozilla.org Graveyard
You need to log in before you can comment on or make changes to this bug.