Ok the commit is reverted :(
On Thu, Jul 2, 2020 at 5:45 PM Murtuza Zabuawala < murtuza.zabuaw...@enterprisedb.com> wrote: > > > > On Thu, Jul 2, 2020 at 5:41 PM Khushboo Vashi < > khushboo.va...@enterprisedb.com> wrote: > >> >> >> On Thu, Jul 2, 2020 at 5:33 PM Murtuza Zabuawala < >> murtuza.zabuaw...@enterprisedb.com> wrote: >> >>> Hi Khushboo, >>> >>> I am already on the latest code :) >>> >> Then your patch will not work. :) >> > > It is applying and I'm able to see the diff, > > ------ > > (venv3_pg) murtuza@ubuntu:~/projects/pgadmin4 (master)$ git apply > browser_regression.diff > (venv3_pg) murtuza@ubuntu:~/projects/pgadmin4 (master)$ gs > On branch master > Your branch is up to date with 'origin/master'. > > Changes not staged for commit: > (use "git add <file>..." to update what will be committed) > (use "git checkout -- <file>..." to discard changes in working directory) > > modified: web/pgadmin/browser/static/js/browser.js > > > ----- > > (venv3_pg) murtuza@ubuntu:~/projects/pgadmin4 (master)$ git diff > diff --git a/web/pgadmin/browser/static/js/browser.js > b/web/pgadmin/browser/static/js/browser.js > index 9db31de59..dc6c26920 100644 > --- a/web/pgadmin/browser/static/js/browser.js > +++ b/web/pgadmin/browser/static/js/browser.js > @@ -278,16 +278,16 @@ define('pgadmin.browser', [ > /* Add hooked-in panels by extensions */ > var panels = JSON.parse(pgBrowser.panels_items); > _.each(panels, function(panel) { > - if (panel.isIframe) { > + if (panel.is_iframe) { > pgBrowser.frames[panel.name] = new pgBrowser.Frame({ > name: panel.name, > title: panel.title, > icon: panel.icon, > width: panel.width, > height: panel.height, > - showTitle: panel.showTitle, > - isCloseable: panel.isCloseable, > - isPrivate: panel.isPrivate, > + showTitle: panel.show_title, > + isCloseable: panel.is_closeable, > + isPrivate: panel.is_private, > url: panel.content, > }); > } else { > @@ -297,12 +297,12 @@ define('pgadmin.browser', [ > icon: panel.icon, > width: panel.width, > height: panel.height, > - showTitle: panel.showTitle, > - isCloseable: panel.isCloseable, > - isPrivate: panel.isPrivate, > + showTitle: panel.show_title, > + isCloseable: panel.is_closeable, > + isPrivate: panel.is_private, > content: (panel.content) ? panel.content : '', > events: (panel.events) ? panel.events : '', > - canHide: (panel.canHide) ? panel.canHide : '', > + canHide: (panel.can_hide) ? panel.can_hide : '', > }); > } > }); > > > > >> >>> *commit* ec30e6ace78a813eb5215efc5c1f0e94582ffc12 (HEAD -> master, >>> origin/master, origin/HEAD) >>> >>> >>> -- >>> Regards, >>> Murtuza Zabuawala >>> *EDB* >>> *POWER TO POSTGRES* >>> https://www.edbpostgres.com >>> >>> >>> On Thu, Jul 2, 2020 at 5:25 PM Khushboo Vashi < >>> khushboo.va...@enterprisedb.com> wrote: >>> >>>> >>>> >>>> On Thu, Jul 2, 2020 at 5:16 PM Murtuza Zabuawala < >>>> murtuza.zabuaw...@enterprisedb.com> wrote: >>>> >>>>> Hi, >>>>> >>>>> There was a regression where we have changed the panel variables in >>>>> menu.py but we did not change their respective reference in JS which >>>>> is causing issues while loading wcDocker Panel/Frames which are defined in >>>>> python side. >>>>> >>>>> This has been fixed, please get the latest pgAdmin code. >>>> >>>>> Ref: 830ac1cc9177c8b9ebed2304924cec1db22e45c3 >>>>> <https://git.postgresql.org/gitweb/?p=pgadmin4.git;a=commit;h=830ac1cc9177c8b9ebed2304924cec1db22e45c3> >>>>> >>>>> >>>>> -- >>>>> Regards, >>>>> Murtuza Zabuawala >>>>> *EDB* >>>>> *POWER TO POSTGRES* >>>>> https://www.edbpostgres.com >>>>> >>>>