<div dir="ltr">I will try finding time tomorrow. Thanks.<div><br></div><div>Martin</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Nov 29, 2017 at 6:02 PM, Greg Sheremeta <span dir="ltr">&lt;<a href="mailto:gshereme@redhat.com" target="_blank">gshereme@redhat.com</a>&gt;</span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Martin, we&#39;ve merged the fix. (Thanks to Alexander for the super fast fix.)<div><br></div><div>Care to test with master?<br><div class="gmail_extra"><div><div class="h5"><br><div class="gmail_quote">On Wed, Nov 29, 2017 at 10:50 AM, Alexander Wels <span dir="ltr">&lt;<a href="mailto:awels@redhat.com" target="_blank">awels@redhat.com</a>&gt;</span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span>On Wednesday, November 29, 2017 10:16:27 AM EST Martin Sivak wrote:<br>
&gt; Thanks to you too for checking.<br>
&gt;<br>
&gt; Martin<br>
&gt;<br>
<br>
</span>Fix is pending review [1]. We moved the main views to a different class<br>
hierarchy and it wasn&#39;t inheriting the methods to add the buttons.<br>
<br>
Alexander<br>
<br>
[1] <a href="https://gerrit.ovirt.org/c/84901/" rel="noreferrer" target="_blank">https://gerrit.ovirt.org/c/849<wbr>01/</a><br>
<div><div class="m_2947628347571662041h5"><br>
&gt; On Wed, Nov 29, 2017 at 3:07 PM, Greg Sheremeta &lt;<a href="mailto:gshereme@redhat.com" target="_blank">gshereme@redhat.com</a>&gt; wrote:<br>
&gt; &gt; Discussed with Alexander, and yep this is an issue.<br>
&gt; &gt; I opened <a href="https://bugzilla.redhat.com/show_bug.cgi?id=1518724" rel="noreferrer" target="_blank">https://bugzilla.redhat.com/sh<wbr>ow_bug.cgi?id=1518724</a><br>
&gt; &gt;<br>
&gt; &gt; Thanks for checking and reporting!<br>
&gt; &gt;<br>
&gt; &gt; On Wed, Nov 29, 2017 at 8:35 AM, Greg Sheremeta &lt;<a href="mailto:gshereme@redhat.com" target="_blank">gshereme@redhat.com</a>&gt;<br>
&gt; &gt;<br>
&gt; &gt; wrote:<br>
&gt; &gt;&gt; +Alexander, can you take a look?<br>
&gt; &gt;&gt;<br>
&gt; &gt;&gt; On Wed, Nov 29, 2017 at 8:33 AM, Martin Sivak &lt;<a href="mailto:msivak@redhat.com" target="_blank">msivak@redhat.com</a>&gt; wrote:<br>
&gt; &gt;&gt;&gt; Hi,<br>
&gt; &gt;&gt;&gt;<br>
&gt; &gt;&gt;&gt; I think I found one issue.. I do not see the extra menu items that were<br>
&gt; &gt;&gt;&gt; supposed to be added to VMs (used to be available in context menu as<br>
&gt; &gt;&gt;&gt; well).<br>
&gt; &gt;&gt;&gt;<br>
&gt; &gt;&gt;&gt; You can check it with<br>
&gt; &gt;&gt;&gt;<br>
&gt; &gt;&gt;&gt; <a href="http://jenkins.ovirt.org/job/ovirt-optimizer_master_check-pa" rel="noreferrer" target="_blank">http://jenkins.ovirt.org/job/o<wbr>virt-optimizer_master_check-pa</a><br>
&gt; &gt;&gt;&gt; tch-el7-x86_64/65/<br>
&gt; &gt;&gt;&gt;<br>
&gt; &gt;&gt;&gt; Martin<br>
&gt; &gt;&gt;&gt;<br>
&gt; &gt;&gt;&gt; On Wed, Nov 29, 2017 at 1:22 AM, Greg Sheremeta &lt;<a href="mailto:gshereme@redhat.com" target="_blank">gshereme@redhat.com</a>&gt;<br>
&gt; &gt;&gt;&gt;<br>
&gt; &gt;&gt;&gt; wrote:<br>
&gt; &gt;&gt;&gt;&gt; Hey,<br>
&gt; &gt;&gt;&gt;&gt;<br>
&gt; &gt;&gt;&gt;&gt; We haven&#39;t changed the API, so everything should still work. For<br>
&gt; &gt;&gt;&gt;&gt; example, I just installed 4.1 versions of dashboard and support-plugin<br>
&gt; &gt;&gt;&gt;&gt; in<br>
&gt; &gt;&gt;&gt;&gt; master, and all&#39;s well [*]:<br>
&gt; &gt;&gt;&gt;&gt;<br>
&gt; &gt;&gt;&gt;&gt; API:<br>
&gt; &gt;&gt;&gt;&gt;<br>
&gt; &gt;&gt;&gt;&gt; api.addSubTab(&#39;Template&#39;, &#39;Red Hat Documentation&#39;,<br>
&gt; &gt;&gt;&gt;&gt; &#39;my-host-subtab-template&#39;, &#39;&lt;url snip&gt;&#39;, {alignRight: true});<br>
&gt; &gt;&gt;&gt;&gt; api.setTabAccessible(&#39;my-host-<wbr>subtab-template&#39;, true);<br>
&gt; &gt;&gt;&gt;&gt;<br>
&gt; &gt;&gt;&gt;&gt; [image: Inline image 1]<br>
&gt; &gt;&gt;&gt;&gt;<br>
&gt; &gt;&gt;&gt;&gt; [*] however, note that &quot;alignRight&quot; is now ignored<br>
&gt; &gt;&gt;&gt;&gt;<br>
&gt; &gt;&gt;&gt;&gt; And dashboard 4.1.8 installed in master engine:<br>
&gt; &gt;&gt;&gt;&gt;<br>
&gt; &gt;&gt;&gt;&gt; [image: Inline image 2]<br>
&gt; &gt;&gt;&gt;&gt;<br>
&gt; &gt;&gt;&gt;&gt;<br>
&gt; &gt;&gt;&gt;&gt; That said, we probably should have sent some announcement about making<br>
&gt; &gt;&gt;&gt;&gt; sure the plugins actually look good with the new theme. If some plugins<br>
&gt; &gt;&gt;&gt;&gt; are<br>
&gt; &gt;&gt;&gt;&gt; greenish, that will look bad.<br>
&gt; &gt;&gt;&gt;&gt;<br>
&gt; &gt;&gt;&gt;&gt; To help with styling external things, we&#39;ve started exposing more CSS<br>
&gt; &gt;&gt;&gt;&gt; via a new SASS brand module:<br>
&gt; &gt;&gt;&gt;&gt; <a href="https://github.com/oVirt/ovirt-engine/tree/master/frontend/b" rel="noreferrer" target="_blank">https://github.com/oVirt/ovirt<wbr>-engine/tree/master/frontend/b</a><br>
&gt; &gt;&gt;&gt;&gt; rands/ovirt-brand/src/main/sas<wbr>s<br>
&gt; &gt;&gt;&gt;&gt; Also, we&#39;re heavily relying on base PatternFly styles, and plugins<br>
&gt; &gt;&gt;&gt;&gt; should do the same. For example, the new dialog styles are completely<br>
&gt; &gt;&gt;&gt;&gt; PatternFly [<a href="http://www.patternfly.org/pattern-library/forms-and-control" rel="noreferrer" target="_blank">http://www.patternfly.org/pat<wbr>tern-library/forms-and-control</a><br>
&gt; &gt;&gt;&gt;&gt; s/modal-overlay/] and we don&#39;t do anything extra on top of that.<br>
&gt; &gt;&gt;&gt;&gt;<br>
&gt; &gt;&gt;&gt;&gt; Let us know if something doesn&#39;t work, or you need help styling<br>
&gt; &gt;&gt;&gt;&gt; something.<br>
&gt; &gt;&gt;&gt;&gt;<br>
&gt; &gt;&gt;&gt;&gt; Greg<br>
&gt; &gt;&gt;&gt;&gt;<br>
&gt; &gt;&gt;&gt;&gt; On Tue, Nov 28, 2017 at 6:25 PM, Martin Sivak &lt;<a href="mailto:msivak@redhat.com" target="_blank">msivak@redhat.com</a>&gt;<br>
&gt; &gt;&gt;&gt;&gt;<br>
&gt; &gt;&gt;&gt;&gt; wrote:<br>
&gt; &gt;&gt;&gt;&gt;&gt; Hi,<br>
&gt; &gt;&gt;&gt;&gt;&gt;<br>
&gt; &gt;&gt;&gt;&gt;&gt; I just got a question from an ovirt-optimizer user about the support<br>
&gt; &gt;&gt;&gt;&gt;&gt; in 4.2. And I realized I haven&#39;t heard anything about how UI plugins<br>
&gt; &gt;&gt;&gt;&gt;&gt; should be updated to work with the new UI.<br>
&gt; &gt;&gt;&gt;&gt;&gt;<br>
&gt; &gt;&gt;&gt;&gt;&gt; Is there anything special that needs to be done to make the plugin<br>
&gt; &gt;&gt;&gt;&gt;&gt; functional? Or will everything still work somehow?<br>
&gt; &gt;&gt;&gt;&gt;&gt;<br>
&gt; &gt;&gt;&gt;&gt;&gt; Best regards<br>
&gt; &gt;&gt;&gt;&gt;&gt;<br>
&gt; &gt;&gt;&gt;&gt;&gt; Martin Sivak<br>
&gt; &gt;&gt;&gt;&gt;&gt; ______________________________<wbr>_________________<br>
&gt; &gt;&gt;&gt;&gt;&gt; Devel mailing list<br>
&gt; &gt;&gt;&gt;&gt;&gt; <a href="mailto:Devel@ovirt.org" target="_blank">Devel@ovirt.org</a><br>
&gt; &gt;&gt;&gt;&gt;&gt; <a href="http://lists.ovirt.org/mailman/listinfo/devel" rel="noreferrer" target="_blank">http://lists.ovirt.org/mailman<wbr>/listinfo/devel</a><br>
&gt; &gt;&gt;<br>
&gt; &gt;&gt; --<br>
&gt; &gt;&gt;<br>
&gt; &gt;&gt; GREG SHEREMETA<br>
&gt; &gt;&gt;<br>
&gt; &gt;&gt; SENIOR SOFTWARE ENGINEER - TEAM LEAD - RHV UX<br>
&gt; &gt;&gt;<br>
&gt; &gt;&gt; Red Hat NA<br>
&gt; &gt;&gt;<br>
</div></div>&gt; &gt;&gt; &lt;<a href="https://www.redhat.com/" rel="noreferrer" target="_blank">https://www.redhat.com/</a>&gt;<br>
&gt; &gt;&gt;<br>
&gt; &gt;&gt; <a href="mailto:gshereme@redhat.com" target="_blank">gshereme@redhat.com</a>    IRC: gshereme<br>
&gt; &gt;&gt; &lt;<a href="https://red.ht/sig" rel="noreferrer" target="_blank">https://red.ht/sig</a>&gt;<br>
<span>&gt; &gt;<br>
&gt; &gt; --<br>
&gt; &gt;<br>
&gt; &gt; GREG SHEREMETA<br>
&gt; &gt;<br>
&gt; &gt; SENIOR SOFTWARE ENGINEER - TEAM LEAD - RHV UX<br>
&gt; &gt;<br>
&gt; &gt; Red Hat NA<br>
&gt; &gt;<br>
</span>&gt; &gt; &lt;<a href="https://www.redhat.com/" rel="noreferrer" target="_blank">https://www.redhat.com/</a>&gt;<br>
&gt; &gt;<br>
&gt; &gt; <a href="mailto:gshereme@redhat.com" target="_blank">gshereme@redhat.com</a>    IRC: gshereme<br>
&gt; &gt; &lt;<a href="https://red.ht/sig" rel="noreferrer" target="_blank">https://red.ht/sig</a>&gt;<br>
<br>
<br>
</blockquote></div><br><br clear="all"><div><br></div>-- <br></div></div><div class="m_2947628347571662041gmail_signature" data-smartmail="gmail_signature"><div dir="ltr"><div><div dir="ltr"><div><div dir="ltr"><div><div dir="ltr"><div><div><div class="h5"><p style="color:rgb(0,0,0);font-family:overpass,sans-serif;font-weight:bold;margin:0px;padding:0px;font-size:14px;text-transform:uppercase"><span>GREG</span> <span>SHEREMETA</span></p><p style="color:rgb(0,0,0);font-family:overpass,sans-serif;font-size:10px;margin:0px 0px 4px;text-transform:uppercase"><span>SENIOR SOFTWARE ENGINEER - TEAM LEAD - RHV UX</span><span style="color:rgb(170,170,170);margin:0px"></span></p><p style="font-family:overpass,sans-serif;margin:0px;font-size:10px;color:rgb(153,153,153)"><a href="https://www.redhat.com/" style="color:rgb(0,136,206);margin:0px" target="_blank">Red Hat NA<span><br><br></span></a></p></div></div><p style="font-family:overpass,sans-serif;margin:0px 0px 6px;font-size:10px;color:rgb(153,153,153)"><span style="margin:0px;padding:0px"><a href="mailto:gshereme@redhat.com" style="color:rgb(0,136,206);margin:0px" target="_blank">gshereme@redhat.com</a>   </span> <span>IRC: <span>gshereme</span></span></p><table border="0" style="color:rgb(0,0,0);font-family:overpass,sans-serif;font-size:medium"><tbody><tr><td width="100px"><a href="https://red.ht/sig" target="_blank"><img src="https://www.redhat.com/files/brand/email/sig-redhat.png" width="90" height="auto"></a></td></tr></tbody></table></div></div></div></div></div></div></div></div></div>
</div></div></div>
</blockquote></div><br></div>