From: | Dinesh Kumar <dinesh(dot)kumar(at)enterprisedb(dot)com> |
---|---|
To: | Guillaume Lelarge <guillaume(at)lelarge(dot)info> |
Cc: | Dave Page <dpage(at)pgadmin(dot)org>, pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org>, pgAdmin Support <pgadmin-support(at)postgresql(dot)org>, Ashesh Vashi <ashesh(dot)vashi(at)enterprisedb(dot)com> |
Subject: | Re: [pgadmin-hackers] Browser fixes |
Date: | 2013-05-19 15:37:45 |
Message-ID: | CAKWsr7iHB5150BMzoXb7J5=2p+ENeVKQdU1xgYd78viUV2bhcQ@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgadmin-hackers pgadmin-support |
Hi Dave,
Thank you very much for fixing these issue, and i have re-viewed and done
the test cases with this patch and seems now the browser is more stable.
Dinesh
--
*Dinesh Kumar*
Software Engineer
Ph: +918087463317
Skype ID: dinesh.kumar432
www.enterprisedb.co
<http://www.enterprisedb.com/>m<http://www.enterprisedb.com/>
*
Follow us on Twitter*
@EnterpriseDB
Visit EnterpriseDB for tutorials, webinars,
whitepapers<http://www.enterprisedb.com/resources-community> and
more <http://www.enterprisedb.com/resources-community>
On Sat, May 18, 2013 at 6:03 PM, Guillaume Lelarge
<guillaume(at)lelarge(dot)info>wrote:
> On Thu, 2013-05-16 at 20:20 +0100, Dave Page wrote:
> > [CCing to -support as people there have been waiting for some of these
> fixes]
> >
> > The attached patch (against master) cleans up a number of issues in
> > the browser code:
> >
> > - It fixes the collapsing tree issue by reverting patch 1 which
> > attempted to fix a bug caused by patch 2. The bug in patch 2 has been
> > fixed by partially reverting some code that attempted (badly) to
> > re-display objects in the treeview after they had been moved to
> > another schema.
> >
> > - Removes the few "Apply" buttons we had, and all the supporting code
> > for consistency with other dialogues and to dissuade users from
> > editing function code etc. in the dialogues (as we already try to do
> > with a Guru Hint).
> >
> > - The removal of the "Apply" buttons means that patch 3 is no longer
> > required, and has been reverted to simplify the code.
> >
> > - The removal of the "Apply" buttons also obsoletes another bug that
> > caused a crash if renaming an object and then using the Apply button.
> > Dinesh had been working on a patch for this issue which was > 20KB and
> > made the browser code even more complex than it is now.
> >
> > Patch 1:
> http://git.postgresql.org/gitweb/?p=pgadmin3.git;a=commit;h=e97d351dcbc42a38f0054a81f37727d511087ffa
> >
> > Patch 2:
> http://git.postgresql.org/gitweb/?p=pgadmin3.git;a=commit;h=3e05c4d832e5658419094c34ce0ee6959bc1d85a
> >
> > Patch 3:
> http://git.postgresql.org/gitweb/?p=pgadmin3.git;a=commit;h=cb5dba55dc491107e4152bd80ec496294e449934
> >
> > Comments please? I'm planning on applying to master and REL-1_16.
> >
>
> I'm fine with reverting the 3 previous patches, and applying this one.
>
>
> --
> Guillaume
> http://blog.guillaume.lelarge.info
> http://www.dalibo.com
>
>
From | Date | Subject | |
---|---|---|---|
Next Message | Michal Kozusznik | 2013-05-20 09:32:37 | Re: [pgadmin-hackers] Remove dialogue "Apply" buttons? |
Previous Message | Dave Page | 2013-05-18 14:22:16 | Re: Greenplum + pgagent |
From | Date | Subject | |
---|---|---|---|
Next Message | Gabriele Franch | 2013-05-20 09:06:35 | Column comments not shown on 1.18.0 alpha1 |
Previous Message | Guillaume Lelarge | 2013-05-18 13:48:30 | Re: Remove dialogue "Apply" buttons? |