1189761 - Clean up Read All Reviews page

Status: RESOLVED FIXED

(Marketplace Graveyard :: Consumer Pages, defect, P2)

Description

[Chuck Harmston [:chuck]]

- Use hub/spoke header
- Add app name
- Move sign in button to be between review summary and list of reviews
- Move date to between review and action link

Comment 2

[Davor Spasovski [:spasovski]]

https://github.com/mozilla/fireplace/commit/4523258

Comment 3

[Madalin Cotetiu]

(In reply to Chuck Harmston [:chuck] from comment #0)
> - Use hub/spoke header
> - Add app name
> - Move sign in button to be between review summary and list of reviews
> - Move date to between review and action link

Everything looks good except the second thing. The app name is not displayed in Read All Reviews page. Where should the app name be displayed?
Please view this screencast: http://screencast.com/t/hJACVBsjTn

Comment 4

[Davor Spasovski [:spasovski]]

The UI spec wanted it to be as "23 reviews for {App Name}". The reviews API endpoint doesn't provide this info but more importantly we don't have a good way to get this info in an l10n friendly format without hitting the same XSS issue that was related to the "back to {App Name}" bug.

I don't think it's a major issue to ignore that part of the requirement but if others feel strongly we could look at moving the app name in another location.