Merge lp:~donadigo/switchboard-plug-sharing/network-disconnected into lp:~elementary-apps/switchboard-plug-sharing/trunk
Proposed by
Adam Bieńkowski
Status: | Merged | ||||
---|---|---|---|---|---|
Approved by: | Danielle Foré | ||||
Approved revision: | 64 | ||||
Merged at revision: | 61 | ||||
Proposed branch: | lp:~donadigo/switchboard-plug-sharing/network-disconnected | ||||
Merge into: | lp:~elementary-apps/switchboard-plug-sharing/trunk | ||||
Diff against target: |
100 lines (+50/-6) 1 file modified
src/Plug.vala (+50/-6) |
||||
To merge this branch: | bzr merge lp:~donadigo/switchboard-plug-sharing/network-disconnected | ||||
Related bugs: |
|
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Danielle Foré | Needs Fixing | ||
Review via email: mp+297104@code.launchpad.net |
Commit message
* Fix bug #1528346: "Show a message when network is disconnected".
Description of the change
Fixes bug #1528346: "Show a message when network is disconnected".
This branch adds a simple AlertView when the network is disconnected, it uses already available NetworkMonitor from GIO so no need for additional libraries.
* Strings may be a little bit weird, please comment here to propose something better.
* Although the bug involved changing the page status to NOT_AVAILABLE, I've chosen a simpler method to present it directly in the root of the plug content.
To post a comment you must log in.
Looks good except a few things:
For the secondary text maybe "While disconnected from the network, sharing services are not available."
When reconnected, the message is not removed.
I believe that alertview as a built-in link button you can use