Merge lp://staging/~renatofilho/account-plugins/update-owncloud into lp://staging/account-plugins

Proposed by Renato Araujo Oliveira Filho
Status: Merged
Approved by: Alberto Mardegan
Approved revision: 169
Merged at revision: 163
Proposed branch: lp://staging/~renatofilho/account-plugins/update-owncloud
Merge into: lp://staging/account-plugins
Diff against target: 48 lines (+9/-5)
2 files modified
data/providers/owncloud.provider.in.in (+1/-1)
qml/owncloud/NewAccount.qml (+8/-4)
To merge this branch: bzr merge lp://staging/~renatofilho/account-plugins/update-owncloud
Reviewer Review Type Date Requested Status
Alberto Mardegan (community) Approve
Review via email: mp+301551@code.staging.launchpad.net

Commit message

Update owncloud login screen.

Description of the change

It is on silo: 79 if you want to test it.

https://requests.ci-train.ubuntu.com/#/ticket/1643

To post a comment you must log in.
168. By Renato Araujo Oliveira Filho

Updated "placeholderText" for owncloud server url.

Revision history for this message
Alberto Mardegan (mardy) wrote :

Just a tiny fix, see the inline comment.

review: Needs Fixing
169. By Renato Araujo Oliveira Filho

Remove topMargin set property.

Revision history for this message
Renato Araujo Oliveira Filho (renatofilho) wrote :

> Just a tiny fix, see the inline comment.
Fixed at Rev. 169

Revision history for this message
Alberto Mardegan (mardy) wrote :

LGTM!

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches