<table><tr><td style="">vpilo 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/D18621">View Revision</a></tr></table><br /><div><div><blockquote style="border-left: 3px solid #8C98B8;
          color: #6B748C;
          font-style: italic;
          margin: 4px 0 12px 0;
          padding: 8px 12px;
          background-color: #F8F9FC;">
<div style="font-style: normal;
          padding-bottom: 4px;">In <a href="https://phabricator.kde.org/D18621#402717" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">D18621#402717</a>, <a href="https://phabricator.kde.org/p/jgrulich/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;">@jgrulich</a> wrote:</div>
<div style="margin: 0;
          padding: 0;
          border: 0;
          color: rgb(107, 116, 140);"><blockquote style="border-left: 3px solid #8C98B8;
          color: #6B748C;
          font-style: italic;
          margin: 4px 0 12px 0;
          padding: 8px 12px;
          background-color: #F8F9FC;">
<div style="font-style: normal;
          padding-bottom: 4px;">In <a href="https://phabricator.kde.org/D18621#402712" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">D18621#402712</a>, <a href="https://phabricator.kde.org/p/vpilo/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;">@vpilo</a> wrote:</div>
<div style="margin: 0;
          padding: 0;
          border: 0;
          color: rgb(107, 116, 140);"><p>But there are legit cases where this timeout is way too long for no reason.</p>

<p>If you close the popup and reopen it before the NM timeout expires, then the first request will fail, no attempt will be made at redoing the scan, and you're stuck for 30 seconds with the same list.</p></div>
</blockquote>

<p>15 seconds.</p></div>
</blockquote>

<p>no, if your device takes 2s for a scan, and if you open the popup, close it and reopen it (causing a new scan request) before the 12s mark, the popup will wait 15 more seconds, totalling ~27s. It was just an extreme example. Regardless, it's still a huge delay, if you consider that any phone will take ~2s for a scan <em>but rescan every 3-4s</em> when you have the wifi settings open.</p>

<blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><p>Then you can fix this instead. You can check last scan attempt and make it to scan a bit later. There is a property in NM::WirelessDevice::LastScan, you can see when the last scan was performed and once the applet is opened, do a scan as soon as possible when you are sure the attempt will not fail.</p></blockquote>

<p>I can find LastScan in the fd.o apidox, but not in NMQt... ?!?</p>

<p>If NM didn't have this timeout, we could have chosen ourselves how to do it (e.g. waiting for LastScan to be 5s in the past, then scan again). But it's now somewhat frustratingly slow, and a button could save a few seconds of annoyance that usually instead result in the user losing their patience and turning wifi off and on instead.<br />
I know this whole proposal sounds silly.. for me this also is silly. This is however about improving the user experience in a few use cases, so I'm trying to mitigate it.</p></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/D18621">https://phabricator.kde.org/D18621</a></div></div><br /><div><strong>To: </strong>vpilo, VDG, Plasma, jgrulich, ngraham<br /><strong>Cc: </strong>alexeymin, plasma-devel, jraleigh, GB_2, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart<br /></div>