<table><tr><td style="">andersonbruce added a comment.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D15093">View Revision</a></tr></table><br /><div><div><p>A couple of notes on the latest upload:</p>

<ol class="remarkup-list">
<li class="remarkup-list-item">I left the background color change code alone. Doing searches on changing widget background color, it looks like the preferred method is to use setPalette() which requires a QPalette, and after trying a couple of things I decided it made more sense to me to pass around two palettes by reference than to pass just the colors and create palettes when they were needed.</li>
</ol>

<ol class="remarkup-list" start="2">
<li class="remarkup-list-item">I backed out the change that used a blank setting rather than the stored values. I agree that it would be nice to not hang on to any extraneous elements if they happened to be created by some other application but what I found was that I had somehow not tested it after making this change and when I did, I found that it had broken the entire "Advanced" widget settings. The problem is that if you start with a blank setting and don't open the Advanced Widget then all the settings in that are lost. To fix this I would have to duplicate a bunch of functionality in both widgets which I don't like doing. Also, if the underlying NetworkManager app adds something I think that it is probably better to not delete it if plasma-nm hasn't been updated yet so I ended up deciding to put it back to the way it was.</li>
</ol></div></div><br /><div><strong>REPOSITORY</strong><div><div>R116 Plasma Network Management Applet</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D15093">https://phabricator.kde.org/D15093</a></div></div><br /><div><strong>To: </strong>andersonbruce, Plasma, jgrulich, pino<br /><strong>Cc: </strong>acrouthamel, K900, pino, lbeltrame, ngraham, plasma-devel, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart<br /></div>