Last Comment Bug 839543 - Add mobile moderated reviews pages
: Add mobile moderated reviews pages
Status: RESOLVED FIXED
p=2
:
Product: Marketplace
Classification: Server Software
Component: Reviewer Tools (show other bugs)
: 1.0
: All All
: P2 normal (vote)
: ---
Assigned To: Davor Spasovski [:spasovski]
:
:
Mentors:
Depends on:
Blocks: 833082
  Show dependency treegraph
 
Reported: 2013-02-08 09:32 PST by Wil Clouser [:clouserw]
Modified: 2013-03-07 19:54 PST (History)
1 user (show)
See Also:
QA Whiteboard:
Iteration: ---
Points: ---


Attachments

Description Wil Clouser [:clouserw] 2013-02-08 09:32:55 PST
This bug is about implementing the moderated review mockup at https://bug834147.bugzilla.mozilla.org/attachment.cgi?id=710535 .  Note moderated reviews here is user entered reviews of apps, not anything to do with app reviewers approving/denying apps.  Yay terminology.

See bug 834147 for the original planning/discussion around this feature.  Bram Pitoyo is point of contact for design questions.

Design notes are below
---------------------------------------------------
In that case, then we can revert the change so that the buttons would perform the function of a radio button, rather than being actionable on tap.

The “Skip for now” is pressed (selected) by default, and user can tap on either the “Keep” or “Reject” buttons to select that option. After done reviewing all options, user can tap on “Process Reviews” to commit all changes.

These selections should have affordance. They need to look like something that can be pressed, so user knows the what he’s supposed to do when he sees it.

Attached is the two visual styles for these: one has a sunken effect, and the other looks it’s been raised.


Bonus: CSS code, below, should we want to implement it later.

.raised {
    box-shadow: inset 0 13px 15px rgba(255,255,255,1);
    background: rgba(225,225,215,0.5);
    color: rgb(164,148,96);
}

.sunken {
    box-shadow: inset 0 0 4px rgba(215,210,195,1);
    color: rgba(0,0,0,0.35);
}
Comment 1 Davor Spasovski [:spasovski] 2013-03-07 19:54:48 PST
done: https://github.com/mozilla/zamboni/commit/61f86c0

Note You need to log in before you can comment on or make changes to this bug.