Merge lp:~cimi/unity8/preview-sharing into lp:unity8
| Status: | Merged |
|---|---|
| Approved by: | Albert Astals Cid on 2015-11-04 |
| Approved revision: | 2005 |
| Merged at revision: | 2045 |
| Proposed branch: | lp:~cimi/unity8/preview-sharing |
| Merge into: | lp:unity8 |
| Prerequisite: | lp:~cimi/unity8/wide-preview-zoomable-videoplayback |
| Diff against target: |
793 lines (+336/-169) 8 files modified
debian/control (+1/-0) po/unity8.pot (+102/-169) qml/Dash/Previews/PreviewMediaToolbar.qml (+36/-0) qml/Dash/Previews/PreviewSharing.qml (+105/-0) qml/Dash/Previews/PreviewVideoPlayback.qml (+10/-0) qml/Dash/Previews/PreviewZoomableImage.qml (+10/-0) tests/qmltests/CMakeLists.txt (+1/-0) tests/qmltests/Dash/Previews/tst_PreviewSharing.qml (+71/-0) |
| To merge this branch: | bzr merge lp:~cimi/unity8/preview-sharing |
| Related bugs: |
| Reviewer | Review Type | Date Requested | Status |
|---|---|---|---|
| PS Jenkins bot | continuous-integration | Needs Fixing on 2015-11-05 | |
| Albert Astals Cid (community) | Approve on 2015-11-04 | ||
| Michał Sawicz | Needs Fixing on 2015-10-26 | ||
| Paweł Stołowski | 2015-10-02 | Approve on 2015-10-13 | |
|
Review via email:
|
|||
Commit Message
Add sharing widget to zoomable image and video playback
Description of the Change
* Are there any related MPs required for this MP to build/function as expected? Please list.
nope, but unity-api needs to document the new scope properties
* Did you perform an exploratory manual test run of your code change and any related functionality?
y
* Did you make sure that your branch does not contain spurious tags?
y
* If you changed the packaging (debian), did you subscribe the ubuntu-unity team to this MP?
n/a
* If you changed the UI, has there been a design review?
y
| PS Jenkins bot (ps-jenkins) wrote : | # |
FAILED: Continuous integration, rev:1996
http://
Executed test runs:
UNSTABLE: http://
UNSTABLE: http://
UNSTABLE: http://
UNSTABLE: http://
SUCCESS: http://
SUCCESS: http://
SUCCESS: http://
SUCCESS: http://
UNSTABLE: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
UNSTABLE: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
Click here to trigger a rebuild:
http://
| PS Jenkins bot (ps-jenkins) wrote : | # |
FAILED: Continuous integration, rev:1997
http://
Executed test runs:
UNSTABLE: http://
UNSTABLE: http://
SUCCESS: http://
SUCCESS: http://
SUCCESS: http://
SUCCESS: http://
SUCCESS: http://
SUCCESS: http://
UNSTABLE: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
UNSTABLE: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
Click here to trigger a rebuild:
http://
| Paweł Stołowski (stolowski) wrote : | # |
Looks good overall, just a couple of inline comments.
| Michael Zanetti (mzanetti) wrote : | # |
Some replies/discussion to Pawel's comments.
| Paweł Stołowski (stolowski) wrote : | # |
See comment inline.
| PS Jenkins bot (ps-jenkins) wrote : | # |
FAILED: Continuous integration, rev:1998
http://
Executed test runs:
UNSTABLE: http://
UNSTABLE: http://
SUCCESS: http://
SUCCESS: http://
SUCCESS: http://
SUCCESS: http://
SUCCESS: http://
SUCCESS: http://
UNSTABLE: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
UNSTABLE: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
Click here to trigger a rebuild:
http://
| Paweł Stołowski (stolowski) wrote : | # |
<pstolowski> kenvandine, hello, can you take a quick look at comments re content type here: https:/
<kenvandine> pstolowski, we want to switch to using mimetypes, but it isn't on the near term roadmap
<pstolowski> kenvandine, if you switch, do you plan to still support the old content types for existing clients (i'm not sure it's really possible though)?
<kenvandine> pstolowski, we'll have to for some time
<kenvandine> have to deprecate it
With that I'm not going to insist on any remapping of mime types (I'm not thrilled by the idea of doing it in the shell, just wanted to rise the concern). We can use c-h content types and when they switch to mime types, update affected scopes (and according to the above, old content types will still be supported for some time before deprecating).
So, it looks good to me, +1. Thanks!
| Albert Astals Cid (aacid) wrote : | # |
sourceComponent: root.visible ? contentPeerComp
Maybe just use the active property for this?
| PS Jenkins bot (ps-jenkins) wrote : | # |
FAILED: Continuous integration, rev:1999
http://
Executed test runs:
UNSTABLE: http://
UNSTABLE: http://
SUCCESS: http://
SUCCESS: http://
SUCCESS: http://
SUCCESS: http://
SUCCESS: http://
SUCCESS: http://
UNSTABLE: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
UNSTABLE: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
Click here to trigger a rebuild:
http://
| Albert Astals Cid (aacid) wrote : | # |
visible: sharingWidget.url != ""
this should probably be sharingWidget.
| Albert Astals Cid (aacid) wrote : | # |
I also have the feeling that it may make sense to make the "toolbar" a reusable component instead of copying the code. What do you think?
| PS Jenkins bot (ps-jenkins) wrote : | # |
FAILED: Continuous integration, rev:2000
http://
Executed test runs:
UNSTABLE: http://
UNSTABLE: http://
SUCCESS: http://
SUCCESS: http://
SUCCESS: http://
SUCCESS: http://
SUCCESS: http://
SUCCESS: http://
UNSTABLE: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
UNSTABLE: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
Click here to trigger a rebuild:
http://
| Albert Astals Cid (aacid) wrote : | # |
* Did you perform an exploratory manual test run of the code change and any related functionality?
Yes
* Did CI run pass? If not, please explain why.
enough
* Did you make sure that the branch does not contain spurious tags?
Yes
| Michał Sawicz (saviq) wrote : | # |
Please bump to 1.3:
/«BUILDDIR»
/«BUILDDIR»
/«BUILDDIR»
/«BUILDDIR»
/«BUILDDIR»
| PS Jenkins bot (ps-jenkins) wrote : | # |
FAILED: Continuous integration, rev:2004
http://
Executed test runs:
SUCCESS: http://
FAILURE: http://
SUCCESS: http://
FAILURE: http://
SUCCESS: http://
SUCCESS: http://
FAILURE: http://
FAILURE: http://
SUCCESS: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
FAILURE: http://
Click here to trigger a rebuild:
http://
| PS Jenkins bot (ps-jenkins) wrote : | # |
FAILED: Continuous integration, rev:2005
http://
Executed test runs:
SUCCESS: http://
FAILURE: http://
SUCCESS: http://
FAILURE: http://
FAILURE: http://
SUCCESS: http://
SUCCESS: http://
FAILURE: http://
SUCCESS: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
SUCCESS: http://
deb: http://
Click here to trigger a rebuild:
http://
| PS Jenkins bot (ps-jenkins) wrote : | # |
FAILED: Continuous integration, rev:2005
http://
Executed test runs:
SUCCESS: http://
FAILURE: http://
SUCCESS: http://
SUCCESS: http://
SUCCESS: http://
SUCCESS: http://
SUCCESS: http://
SUCCESS: http://
SUCCESS: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
SUCCESS: http://
deb: http://
Click here to trigger a rebuild:
http://

FAILED: Continuous integration, rev:1995 jenkins. qa.ubuntu. com/job/ unity8- ci/6411/ jenkins. qa.ubuntu. com/job/ generic- deb-autopilot- vivid-touch/ 4478 jenkins. qa.ubuntu. com/job/ generic- deb-autopilot- wily-touch/ 790 jenkins. qa.ubuntu. com/job/ unity-phablet- qmluitests- vivid/1123 jenkins. qa.ubuntu. com/job/ unity-phablet- qmluitests- wily/442 jenkins. qa.ubuntu. com/job/ unity8- vivid-amd64- ci/1018 jenkins. qa.ubuntu. com/job/ unity8- vivid-i386- ci/1019 jenkins. qa.ubuntu. com/job/ unity8- wily-amd64- ci/650 jenkins. qa.ubuntu. com/job/ unity8- wily-i386- ci/651 jenkins. qa.ubuntu. com/job/ generic- deb-autopilot- runner- vivid-mako/ 3643 jenkins. qa.ubuntu. com/job/ generic- mediumtests- builder- vivid-armhf/ 4475 jenkins. qa.ubuntu. com/job/ generic- mediumtests- builder- vivid-armhf/ 4475/artifact/ work/output/ *zip*/output. zip s-jenkins. ubuntu- ci:8080/ job/touch- flash-device/ 23907 jenkins. qa.ubuntu. com/job/ generic- deb-autopilot- runner- wily-mako/ 477 jenkins. qa.ubuntu. com/job/ generic- mediumtests- builder- wily-armhf/ 790 jenkins. qa.ubuntu. com/job/ generic- mediumtests- builder- wily-armhf/ 790/artifact/ work/output/ *zip*/output. zip s-jenkins. ubuntu- ci:8080/ job/touch- flash-device/ 23908
http://
Executed test runs:
SUCCESS: http://
SUCCESS: http://
UNSTABLE: http://
UNSTABLE: http://
SUCCESS: http://
SUCCESS: http://
SUCCESS: http://
SUCCESS: http://
SUCCESS: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
SUCCESS: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
Click here to trigger a rebuild: s-jenkins. ubuntu- ci:8080/ job/unity8- ci/6411/ rebuild
http://