Review button inactive without explanation

Incomplete until UX gives us the way to go.

Please have a look at the following WF: https://docs.google.com/presentation/d/1-hc17yTx-_Z0gXEAzmpcZfO7GJI86Wh7ko6ujcKtXyg/edit#slide=id.p

Desired behaviour:

- Title of the Rating & review widget to be same size as the others (i.e.: “Info”, “Updates”) and sizes of stars must be x1.5 current size. Contact me for visual specs.
- Once user taps/clicks on the stars to give a rating, the "Add a review" input field will appear, right below, as part as the same widget / area. The ‘Send’ button will remain disabled until user taps/clicks on it to write a review.
- Once user starts typing, the button will change state to active and the rating and review can be submitted.
- OSK behaviour should remain as per current implementation.
- Rating and review display widget should remain as per current implementation.

Also, once the rating and review is submitted and the UI refreshes, it would be desirable that we show 'your just published' review rather that the into at the top, so the user doesn't have to scroll down to see it. Could we set the screen to that reviews area after refresh?

Relatedly, if you rate the app but do not write a review, the Send button is enabled and will let you submit it (which is not intended, see comment #3).

The scope will refresh, but you won't see your review! Because (I assume) the review server rejected the textless review.

This bug seems to appear because the click scope specifies "required" = "review" for preview data [1]. If it specified nothing, both review and rating would be required, AFAIK.

So there is a further fix in the click scope to drop those lines I believe. I've added a task.

[1] http://bazaar.launchpad.net/~ubuntuone-control-tower/unity-scope-click/trunk/view/head:/libclickscope/click/preview.cpp#L934

(sorry, small typo in my comment #5. The click scope currently specifies "required" = "rating".)

Err, that request might be a duplicate of bug 1580935.

@mterry It is, so I've deleted the task from this bug, as it's a separate issue.

This bug was fixed in the package unity8 - 8.12+16.10.20160527-0ubuntu1

---------------
unity8 (8.12+16.10.20160527-0ubuntu1) yakkety; urgency=medium

  [ Albert Astals Cid ]
  * Activate on click for SimpleMessages (LP: #1421696)
  * Add override
  * Change the art back to the cardData value if it changes from broken
    to valid image (LP: #1533577)
  * Concierge mode generated code optimization
  * Don't use context properties but properties of the delegates
  * Listen to contentItem::widthChanged instead of
    Flickable::contentWidthChanged (LP: #1565763)
  * Make some AP code faster
  * Make tests a bit more stable
  * Make unity8 and unity8-dash handle termination signals
  * Remove workaround not needed anymore (LP: #1475643)
  * Resolve cardArtStyle on compile time
  * Set the theme earlier (LP: #1574048)

  [ Albert Astals Cid, Daniel d'Andrada ]
  * Remove DirectionalDragArea and libs/UbuntuGestures and port to SDK
    equivalents

  [ Andrea Cimitan ]
  * Hide the preview review input field when a rating is required (LP:
    #1541971)

  [ CI Train Bot ]
  * Resync trunk.

  [ Daniel d'Andrada ]
  * Fix DragHandle so it works in all directions
  * Support animated cursors
  * WindowInputMonitor - also map Qt::Key_HomePage to our home key
  * plugins/Cursor: properly register to the screen's QPlatformCursor
    when screen changes (LP: #1579742)

  [ Josh Arenson ]
  * Close the PageHeaderExtraPanel when a filter option is selected (LP:
    #1569498)
  * Wrap primaryFilter in a flickable to make it behave nicely. (LP:
    #1569492)

  [ Lukáš Tinkl ]
  * First snap decision should always be expanded, unless user decides
    otherwise (LP: #1580090, #1575045)
  * Fullscreen notification bug fixes (LP: #1583944, #1581498, #1422711)
  * Implement maximizing windows horizontally/vertically
  * Lock the session when putting the laptop into sleep (LP: #1581063)

  [ Lukáš Tinkl, Michael Terry ]
  * With a maximized window, "dragging" the panel down should restore it

  [ Michael Terry ]
  * Change some dash button colors from orange to green. (LP: #1581047)
  * Use PageHeader instead Ambiance's PageHeadStyle
  * Use new setTimeZone method to set user-friendly timezone name in the
    wizard. (LP: #1566295)

  [ Michael Zanetti ]
  * Remove the background dimming when the launcher is revealed by a
    drag (LP: #1575137)
  * reenable or drop disabled tests

 -- Michael Terry <email address hidden> Fri, 27 May 2016 13:54:44 +0000