<table><tr><td style="">dvratil requested changes to this revision.<br />dvratil added a comment.<br />This revision now requires changes to proceed.
</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/D10896" rel="noreferrer">View Revision</a></tr></table><br /><div><div><p>Hmm, I guess you can now kill <tt style="background: #ebebeb; font-size: 13px;">m_buttonTimer</tt>, <tt style="background: #ebebeb; font-size: 13px;">m_iteration</tt>,  <tt style="background: #ebebeb; font-size: 13px;">resetDisplaySwitch()</tt> and <tt style="background: #ebebeb; font-size: 13px;">applyGenericConfig()</tt> as well, as those are used exclusively by the config iteration. You probably also want to check if the Selection OSD is already shown, otherwise pressing the Display Button multiple times would create multiple OSDs.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R104 KScreen</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D10896" rel="noreferrer">https://phabricator.kde.org/D10896</a></div></div><br /><div><strong>To: </strong>sebas, Plasma, dvratil<br /><strong>Cc: </strong>plasma-devel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart<br /></div>