Review Request: Make Network Manager show signal strength in system tray also for mobile broadband connections

[Lamarque Vieira Souza]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/102827/#review7264
-----------------------------------------------------------



applet/networkmanager.cpp
<http://git.reviewboard.kde.org/r/102827/#comment6320>

    remove extra space.



applet/networkmanager.cpp
<http://git.reviewboard.kde.org/r/102827/#comment6332>

    this else clause is not necessary.



applet/networkmanager.cpp
<http://git.reviewboard.kde.org/r/102827/#comment6328>

    add a "else" after the "}" to prevent testing the modem clause when the wifi one has already been used.



applet/networkmanager.cpp
<http://git.reviewboard.kde.org/r/102827/#comment6322>

    remove extra space.



applet/networkmanager.cpp
<http://git.reviewboard.kde.org/r/102827/#comment6323>

    remote extra space.



applet/networkmanager.cpp
<http://git.reviewboard.kde.org/r/102827/#comment6324>

    remove extra space.



applet/networkmanager.cpp
<http://git.reviewboard.kde.org/r/102827/#comment6321>

    remote extra space.



applet/networkmanager.cpp
<http://git.reviewboard.kde.org/r/102827/#comment6326>

    remote extra space.



applet/networkmanager.cpp
<http://git.reviewboard.kde.org/r/102827/#comment6329>

    use the variable accesstechnology instead of calling this method again.



applet/networkmanager.cpp
<http://git.reviewboard.kde.org/r/102827/#comment6327>

    remove extra space.



applet/networkmanager.cpp
<http://git.reviewboard.kde.org/r/102827/#comment6330>

    optimization: remove this else clause.



applet/networkmanager.cpp
<http://git.reviewboard.kde.org/r/102827/#comment6331>

    optimization: remove this else clause and just add 'return QString("phone");' incondicionally.



applet/networkmanager.cpp
<http://git.reviewboard.kde.org/r/102827/#comment6337>

    I do not think there is a "phone" element in the network.svgz. It must be added for this to work, unless Plasma::Svg is smart enough to search for pixmaps when there is no element.



applet/networkmanager.cpp
<http://git.reviewboard.kde.org/r/102827/#comment6325>

    remove extra space.



applet/networkmanager.cpp
<http://git.reviewboard.kde.org/r/102827/#comment6333>

    This test is required to prevent crashes with CDMA phones.



applet/networkmanager.cpp
<http://git.reviewboard.kde.org/r/102827/#comment6334>

    signal signalQualityChanged passes an uint variable, not int. The QObject::connect does not work because of that.



applet/networkmanager.cpp
<http://git.reviewboard.kde.org/r/102827/#comment6336>

    accessTechnologyChanges passes a variable of type Solid::Control::ModemInterface::AccessTechnology, not int. The QObject::connect does not work because of that.



libs/internals/uiutils.cpp
<http://git.reviewboard.kde.org/r/102827/#comment6335>

    This comment is not necessary, remove it. Uutils::iconName() is for pixmap icons only, not svg.


- Lamarque Vieira Souza


On Oct. 11, 2011, 4:05 p.m., Kai Uwe Broulik wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/102827/
> -----------------------------------------------------------
> 
> (Updated Oct. 11, 2011, 4:05 p.m.)
> 
> 
> Review request for Network Management.
> 
> 
> Description
> -------
> 
> As I now use a Surfstick to get to the internet, I found it handy to see the signal strength right in system tray like WiFi connections do, rather than opening the popup all the time.
> This patch resolves this issue by making the system tray show the signal strength as well as the access technology (Edge, umts, etc) in a similar way like Plasma Active's (fake?) mobilesignal widget does, where I also borrowed the graphics from :P
> I currently only respected the Plasma SVG icon case, i.e. when you do not use SVG graphics, it uses the generic phone icon still, as we do not yet have a icon naming spec for such cases and do not have icons either, so the tooltip and popup have the generic phone icon.
> 
> I added the HSDPA and 3G strength icons to the network.svgz icon file from network manager. See attachments.
> 
> 
> This addresses bug 283776.
>     http://bugs.kde.org/show_bug.cgi?id=283776
> 
> 
> Diffs
> -----
> 
>   applet/networkmanager.cpp 73a9cf4 
>   libs/internals/uiutils.cpp f90e44e 
> 
> Diff: http://git.reviewboard.kde.org/r/102827/diff/diff
> 
> 
> Testing
> -------
> 
> Compiles and works okay.
> It is not yet perfect but I think it is a good start.
> 
> Known issues:
>  - The strength indicator is only updated if you click (hover?) the system tray icon. The konsole output states that the signals "Solid::Control::ModemGsmNetworkInterface::signalQualityChanged()" do not exist (but I don't know why?) and thus not being triggered
>  - When not connected, there is either no icon at all (I wanted to use the no-wifi connected icon as fallback) or the 3G with 50% strength (this is weird)
> 
> 
> Screenshots
> -----------
> 
> HSDPA tray icon
>   http://git.reviewboard.kde.org/r/102827/s/289/
> 
> 
> Thanks,
> 
> Kai Uwe Broulik
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-networkmanager/attachments/20111012/7894fae7/attachment-0001.html>