<div dir='auto'><div>Yeah, I'll 1+ the patch. The move is an easy one once the bug is resolved.</div><div dir="auto"><br></div><div dir="auto">I'm quite amused by the bug, and that someone else filed the same bug a couple of hours right before me (and I missed it). Will review his fix.</div><div dir="auto"><br></div><div dir="auto">Looking forward to your other two patches.<br><div class="gmail_extra" dir="auto"><br><div class="gmail_quote">On 26/01/2017 9:37 PM, Rob Cresswell <robert.cresswell@outlook.com> wrote:<br type="attribution"><blockquote class="quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">



<div>
<div dir="ltr">Wow, lots of replies. Thanks for the FFE. I'd request that we proceed with the panel separation now, and resolve the final location when any bug fixes have merged. I don't really want to hold up a whole chain of patches over a one line path change.
 API Access and Key Pairs are done, and I'll put up Security Groups and Floating IPs once they start moving (maintaining huge patch chains is a waste of time)
<div><br>
</div>
<div>Rob</div>
</div>
<div><br>
<div class="elided-text">On 26 January 2017 at 02:09, Adrian Turjak <span dir="ltr">
<<a href="mailto:adriant@catalyst.net.nz">adriant@catalyst.net.nz</a>></span> wrote:<br>
<blockquote style="margin:0 0 0 0.8ex;border-left:1px #ccc solid;padding-left:1ex">
<div>I've posted some comments on the API Access patch.<br>
<p><br>
</p>
The blueprint was saying that 'API Access' would be both at the Project level, but the way panel groups worked meant that setting the 'default' panel group didn't work when that dashboard already had panel groups, since the default panel group was annoyingly
 hidden away because of somewhat odd template logic.<br>
<br>
I submitted a bug report here:<br>
<a href="https://bugs.launchpad.net/horizon/+bug/1659456">https://bugs.launchpad.net/<wbr>horizon/+bug/1659456</a><br>
<br>
And proposed a fix for that here:<br>
<a href="https://review.openstack.org/#/c/425486">https://review.openstack.org/#<wbr>/c/425486</a><br>
<br>
With that change the default group panels are not hidden, and displayed at the same level as the other panel groups. This then allows us to move API Access to the top level where the blueprint says. This makes much more sense since API Access isn't a compute
 only thing.
<div>
<div><br>
<br>
<div>On 26/01/17 12:02, Fox, Kevin M wrote:<br>
</div>
<blockquote>
<div style="direction:ltr;font-family:'tahoma';color:#000000;font-size:10pt">Big Thanks! from me too. The old UI here was very unintuitive, so I had to field a lot of questions related to it. This is great. :)<br>
<br>
Kevin<br>
<div style="font-family:'times new roman';color:#000000;font-size:16px">
<hr>
<div style="direction:ltr"><font color="#000000" face="Tahoma" size="2"><b>From:</b> Lingxian Kong [<a href="mailto:anlin.kong@gmail.com">anlin.kong@gmail.com</a>]<br>
<b>Sent:</b> Wednesday, January 25, 2017 2:23 PM<br>
<b>To:</b> OpenStack Development Mailing List (not for usage questions)<br>
<b>Subject:</b> Re: [openstack-dev] [horizon] FFE Request<br>
</font><br>
</div>
<div>
<div dir="ltr">
<div style="font-family:monospace , monospace">Hi, Rob,</div>
<div style="font-family:monospace , monospace"><br>
</div>
<div style="font-family:monospace , monospace">First, thanks for your work!</div>
<div style="font-family:monospace , monospace"><br>
</div>
<div style="font-family:monospace , monospace">What's your plan for the other two tabs (security group, floatingip)? I could see the split is very helpful no matter from performance perspective and both useful from end user's perspective.</div>
<div style="font-family:monospace , monospace"><br>
</div>
<div style="font-family:monospace , monospace">BTW, a huge +1 for this FFE!</div>
<div style="font-family:monospace , monospace"><br>
</div>
<div style="font-family:monospace , monospace"><br>
</div>
</div>
<div><br clear="all">
<div>
<div>
<div dir="ltr">
<div>
<div dir="ltr">
<div><font face="monospace, monospace"><br>
Cheers,<br>
Lingxian Kong (Larry)</font></div>
</div>
</div>
</div>
</div>
</div>
<br>
<div class="elided-text">On Thu, Jan 26, 2017 at 9:01 AM, Adrian Turjak <span dir="ltr">
<<a href="mailto:adriant@catalyst.net.nz">adriant@catalyst.net.nz</a>></span> wrote:<br>
<blockquote style="margin:0 0 0 0.8ex;border-left:1px #ccc solid;padding-left:1ex">
<div dir="auto">
<div>+1
<div dir="auto"><br>
</div>
<div dir="auto">We very much need this as the performance of that panel is awful. This solves that problem while being a fairly minor code change which also provides much better UX.</div>
<div>
<div><br>
<div><br>
<div class="elided-text">On 26/01/2017 8:07 AM, Rob Cresswell <<a href="mailto:robert.cresswell@outlook.com"></a><a href="mailto:robert.cresswell@outlook.com">robert.cresswell@outlook.com</a>>
 wrote:<br>
<blockquote style="margin:0 0 0 0.8ex;border-left:1px #ccc solid;padding-left:1ex">
<div>
<div dir="ltr">o/ 
<div><br>
</div>
<div>I'd like to request an FFE on <a href="https://blueprints.launchpad.net/horizon/+spec/reorganise-access-and-security"></a><a href="https://blueprints">https://blueprints</a>.<a href="http://launchpad.net/horizon/+spec/">launchpa<wbr>d.net/horizon/+spec/</a>reorganise<wbr>-access-and-security.
 This blueprint splits up the access and security tabs into 4 distinct panels. The first two patches are <a href="https://review.openstack.org/#/c/408247"></a><a href="https://review.openstack">https://review.openstack</a>.o<wbr>rg/#/c/408247
 and <a href="https://review.openstack.org/#/c/425345/">https://review.openstack.o<wbr>rg/#/c/425345/</a> </div>
<div><br>
</div>
<div>It's low risk; no API layer changes, mostly just moving code around.</div>
<div><br>
</div>
<div>Rob</div>
</div>
</div>
</blockquote>
</div>
<br>
</div>
</div>
</div>
</div>
</div>
<br>
______________________________<wbr>______________________________<wbr>______________<br>
OpenStack Development Mailing List (not for usage questions)<br>
Unsubscribe: <a href="http://OpenStack-dev-request@lists.openstack.org?subject:unsubscribe">
OpenStack-dev-request@lists.op<wbr>enstack.org?subject:unsubscrib<wbr>e</a><br>
<a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev">http://lists.openstack.org/cgi<wbr>-bin/mailman/listinfo/openstac<wbr>k-dev</a><br>
<br>
</blockquote>
</div>
<br>
</div>
</div>
</div>
</div>
<br>
<fieldset></fieldset> <br>
<pre>______________________________<wbr>______________________________<wbr>______________
OpenStack Development Mailing List (not for usage questions)
Unsubscribe: <a href="mailto:OpenStack-dev-request@lists.openstack.org?subject:unsubscribe">OpenStack-dev-request@lists.<wbr>openstack.org?subject:<wbr>unsubscribe</a>
<a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev">http://lists.openstack.org/<wbr>cgi-bin/mailman/listinfo/<wbr>openstack-dev</a>
</pre>
</blockquote>
<br>
</div>
</div>
</div>
</blockquote>
</div>
<br>
</div>
</div>

</blockquote></div><br></div></div></div>