<table><tr><td style="">davidedmundson retitled this revision from "Alt fix for D3407" to "Don't set PanelView visibilty when opening/closing config
".<br />davidedmundson updated the summary for this revision. <a href="https://phabricator.kde.org/transactions/detail/PHID-XACT-DREV-7rqn4wnglev7hp3/" rel="noreferrer">(Show Details)</a><br />davidedmundson updated the test plan for this revision. <a href="https://phabricator.kde.org/transactions/detail/PHID-XACT-DREV-ielgncn4p7eto7y/" rel="noreferrer">(Show Details)</a>
</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/D3408" rel="noreferrer">View Revision</a></tr></table><br /><div><strong>EDIT DETAILS</strong><div><div style="white-space: pre-wrap; color: #74777D;"><span style="padding: 0 2px; color: #802b2b; background: rgba(251, 175, 175, .7);">Had the code in my head having reviewed it,</span><span style="padding: 0 2px; color: #3e6d35; background: rgba(151, 234, 151, .6);">    Instead of making PanelViewConfig manipulate the visibiltyMode of<br />
    PanelView, make PanelView force the view to be visible whilst the config<br />
    is open.<br />
    <br />
    This is cleaner as</span> <span style="padding: 0 2px; color: #802b2b; background: rgba(251, 175, 175, .7);">so wanted to post what I<br />
meant</span><span style="padding: 0 2px; color: #3e6d35; background: rgba(151, 234, 151, .6);">it doesn't alter the original config,</span> <span style="padding: 0 2px; color: #802b2b; background: rgba(251, 175, 175, .7);">before I forgot.<br />
<br />
Don't merge yet, I somehow ended up reversing a giant if statement logic and need to check it works.</span><span style="padding: 0 2px; color: #3e6d35; background: rgba(151, 234, 151, .6);">and resolves a<br />
    bug that opening configure with an autohide panel would shift contents<br />
    about.<br />
</span></div></div></div><br /><div><strong>EDIT DETAILS</strong><div><div style="white-space: pre-wrap; color: #74777D;"><span style="padding: 0 2px; color: #3e6d35; background: rgba(151, 234, 151, .6);">With panel as autohide opened config<br />
Moused away from panel, panel stayed visible<br />
<br />
With panel as autohide, plugged in a USB pen - panel appeared - and closed when I hid the dialog<br />
<br />
Changed mode in panel config, panel didn't immediately change - but did on close.<br />
Tested with all modes.</span></div></div></div><br /><div><strong>REPOSITORY</strong><div><div>rPLASMAWORKSPACE Plasma Workspace</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D3408" rel="noreferrer">https://phabricator.kde.org/D3408</a></div></div><br /><div><strong>EMAIL PREFERENCES</strong><div><a href="https://phabricator.kde.org/settings/panel/emailpreferences/" rel="noreferrer">https://phabricator.kde.org/settings/panel/emailpreferences/</a></div></div><br /><div><strong>To: </strong>davidedmundson, Plasma<br /><strong>Cc: </strong>mart, plasma-devel, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas<br /></div>