Merge lp:~pete-woods/indicator-network/lp1541588 into lp:indicator-network/15.10

Proposed by Pete Woods
Status: Merged
Approved by: Pete Woods
Approved revision: 559
Merged at revision: 559
Proposed branch: lp:~pete-woods/indicator-network/lp1541588
Merge into: lp:indicator-network/15.10
Prerequisite: lp:~pete-woods/indicator-network/ofono-optional
Diff against target: 12 lines (+1/-1)
1 file modified
src/indicator/sections/wwan-section.cpp (+1/-1)
To merge this branch: bzr merge lp:~pete-woods/indicator-network/lp1541588
Reviewer Review Type Date Requested Status
Indicator Applet Developers Pending
Review via email: mp+285881@code.launchpad.net

This proposal supersedes a proposal from 2016-02-12.

Commit message

Only show the modem menu when there's actually a modem

Description of the change

Only show the modem menu when there's actually a modem

To post a comment you must log in.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'src/indicator/sections/wwan-section.cpp'
2--- src/indicator/sections/wwan-section.cpp 2015-10-06 10:19:30 +0000
3+++ src/indicator/sections/wwan-section.cpp 2016-02-12 14:25:38 +0000
4@@ -147,7 +147,7 @@
5 m_linkMenuMerger->append(pair.second->menuModel());
6 }
7
8- if (modems.size() == -1)
9+ if (modems.size() == 0)
10 {
11 m_bottomMenu->clear();
12 }

Subscribers

People subscribed via source and target branches