----------------------------------------------------------- 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 > >
_______________________________________________ kde-networkmanager mailing list [email protected] https://mail.kde.org/mailman/listinfo/kde-networkmanager
