<div dir="ltr">On Tue, Aug 27, 2013 at 7:00 AM, Kasun Chathuranga <span dir="ltr"><<a href="mailto:chathuranga.jayaneththi@gmail.com" target="_blank">chathuranga.jayaneththi@gmail.com</a>></span> wrote:<br><div class="gmail_extra">
<div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div dir="ltr"><br><div class="gmail_extra">
<div class="im"><br><br><div class="gmail_quote">On Mon, Aug 26, 2013 at 10:03 PM, Isaac Bennetch <span dir="ltr"><<a href="mailto:bennetch@gmail.com" target="_blank">bennetch@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;border-left-color:rgb(204,204,204);padding-left:1ex"><div><br>
<br>
On 8/26/13 9:42 AM, Marc Delisle wrote:<br>
> Hi,<br>
> please have a look at the current state of QA_4_0. I encouraged Atul to<br>
> merge his implementation for RFE 1435 and he did some other improvements<br>
> to the navi panel.<br>
><br>
> I plan to release 4.0.6-rc1 tomorrow with these changes.<br>
<br>
</div>Looks good and your plan sounds good.<br>
<br>
>From the user feedback, I think there is still room for improvement<br>
here, but as it's not inconvenient for me personally it's difficult for<br>
me to say how exactly it can be improved. It works well for me.<br>
<div><div><br>
</div></div></blockquote></div></div>An improvement that can added is opening databases when they are grouped (possibly a couple of levels down) by the database name prefix. This does not seem to be handled in the current implementation.<br>
<br></div></div></blockquote><div><div style="font-family:arial,sans-serif;font-size:13px">The pull request is 4 days old, looks like we are doubly implementing the same thing. About expanding to multiple levels, I decided to move further only after some feedback.</div>
<div style="font-family:arial,sans-serif;font-size:13px">But now it looks like you have already done that, though I have not seen your implementation working as it seems quite a conflicting merge into master...</div><div style="font-family:arial,sans-serif;font-size:13px">
<br></div></div><div style="font-family:arial,sans-serif;font-size:13px"><br></div></div>-- <br>Atul Pratap Singh<div><a href="http://blog.atulsisodia.in/" target="_blank">http://blog.atulsisodia.in/</a></div>
</div></div>