Re: [pgAdmin][RM4351] Confirm before close of properties dialog if changes not saved
Aditya Toshniwal <[email protected]>
| Newsgroups | gmane.comp.db.postgresql.pgadmin.devel |
|---|---|
| Message-ID | <CAM9w-_kFtokCkMnsgG4r9XLcgZc4KJcapXkJC7C+9atcZ0cBLw@mail.gmail.com> |
On Wed, Oct 30, 2019 at 3:44 PM Dave Page <[email protected]> wrote: > Hi > > On Wed, Oct 30, 2019 at 4:40 AM Aditya Toshniwal < > [email protected]> wrote: > >> Hi, >> >> On Tue, Oct 29, 2019 at 9:01 PM Dave Page <[email protected]> wrote: >> >>> Hi >>> >>> I just found an, umm, embarrassing issue with this. It asks for the >>> cancel confirmation even when pressing the OK button :-o >>> >> Tested all but missed the obvious. Bad miss :/ >> Attached is the patch to fix the issue. >> > > Thanks - applied. Should we also warn on Reset? That seems logical to me. > Can be added. Preference option should be common for both Cancel and Reset. Will send an add-on patch for this. > > >> >>> Can you fix that ASAP please? >>> >>> On Tue, Oct 29, 2019 at 2:35 PM Dave Page <[email protected]> wrote: >>> >>>> Thanks, applied. >>>> >>>> On Tue, Oct 29, 2019 at 6:40 AM Aditya Toshniwal < >>>> [email protected]> wrote: >>>> >>>>> Hi Hackers, >>>>> >>>>> Attached is the patch to warn the user before closing an unsaved >>>>> properties dialog. >>>>> Additionally, I have added a preference option in Browser->Display >>>>> where the user can disable this. Default is enabled. >>>>> Docs updated. >>>>> >>>>> Kindly review. >>>>> >>>>> -- >>>>> Thanks and Regards, >>>>> Aditya Toshniwal >>>>> Sr. Software Engineer | EnterpriseDB India | Pune >>>>> "Don't Complain about Heat, Plant a TREE" >>>>> >>>> >>>> >>>> -- >>>> Dave Page >>>> Blog: http://pgsnake.blogspot.com >>>> Twitter: @pgsnake >>>> >>>> EnterpriseDB UK: http://www.enterprisedb.com >>>> The Enterprise PostgreSQL Company >>>> >>> >>> >>> -- >>> Dave Page >>> Blog: http://pgsnake.blogspot.com >>> Twitter: @pgsnake >>> >>> EnterpriseDB UK: http://www.enterprisedb.com >>> The Enterprise PostgreSQL Company >>> >> >> >> -- >> Thanks and Regards, >> Aditya Toshniwal >> Sr. Software Engineer | EnterpriseDB India | Pune >> "Don't Complain about Heat, Plant a TREE" >> > > > -- > Dave Page > Blog: http://pgsnake.blogspot.com > Twitter: @pgsnake > > EnterpriseDB UK: http://www.enterprisedb.com > The Enterprise PostgreSQL Company > -- Thanks and Regards, Aditya Toshniwal Sr. Software Engineer | EnterpriseDB India | Pune "Don't Complain about Heat, Plant a TREE"