Merge lp:~andrewsomething/ubuntu-system-settings-online-accounts/1235004 into lp:ubuntu-system-settings-online-accounts
Proposed by
Andrew Starr-Bochicchio
Status: | Superseded | ||||
---|---|---|---|---|---|
Proposed branch: | lp:~andrewsomething/ubuntu-system-settings-online-accounts/1235004 | ||||
Merge into: | lp:ubuntu-system-settings-online-accounts | ||||
Diff against target: |
64 lines (+34/-0) 1 file modified
src/module/OAuth.qml (+34/-0) |
||||
To merge this branch: | bzr merge lp:~andrewsomething/ubuntu-system-settings-online-accounts/1235004 | ||||
Related bugs: |
|
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Ken VanDine | Needs Fixing | ||
Review via email: mp+189397@code.launchpad.net |
This proposal has been superseded by a proposal from 2013-10-08.
Commit message
Add loading indicator to OAuth page. Fixes: https:/
To post a comment you must log in.
This looks pretty good, but you can simplify the visible property. Instead of:
visible: loading = true
Since loading is a bool, you can just do this:
visible: loading