<table><tr><td style="">mvourlakos 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/D18918">View Revision</a></tr></table><br /><div><div><p><a href="https://phabricator.kde.org/p/trmdi/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;">@trmdi</a> please do me a favour:</p>

<ol class="remarkup-list">
<li class="remarkup-list-item">close this review and open a clean new one because for some reason when trying to test it in my system includes also one old commit that I added the ViewType option</li>
<li class="remarkup-list-item">no problem with & characters you can add them</li>
<li class="remarkup-list-item">no problem to hide the Dock/Panel View when (configuring)</li>
<li class="remarkup-list-item">please dont use the config file to access the viewType, I found an alternative way that it is in the master, we sniff now the viewType through dbus contextMenuData. The menu sends the containment->id() and when contextMenuData are requested these data include also the view type.</li>
<li class="remarkup-list-item">For the setVisible(true) and how this bug occurs we will discuss it again at the new review</li>
</ol></div></div><br /><div><strong>REPOSITORY</strong><div><div>R878 Latte Dock</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D18918">https://phabricator.kde.org/D18918</a></div></div><br /><div><strong>To: </strong>trmdi, mvourlakos<br /><strong>Cc: </strong>plasma-devel, jraleigh, GB_2, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart<br /></div>