Re: Code Folding in Code Mirror

2017-08-15 Thread Sarah McAlear
Ok, it does work for the cases defined. We misunderstood how it works. Still, could we enable this everywhere? What is the reason not to? Thanks, Bing & Sarah On Tue, Aug 15, 2017 at 5:00 PM, Sarah McAlear <smcal...@pivotal.io> wrote: > Hi Hackers! > > We are working on unif

Code Folding in Code Mirror

2017-08-15 Thread Sarah McAlear
Hi Hackers! We are working on unifying the CodeMirror stuff into one location. While looking at this we noticed that while there is logic for code folding, it doesn't seem to be working. Does it work for everyone else? We also started to wonder if there is a specific reason that code folding

Re: [pgAdmin4][RM2586] Cleanup feature test

2017-08-11 Thread Sarah McAlear
Harshal > > -- > *Harshal Dhumal* > *Sr. Software Engineer* > > EnterpriseDB India: http://www.enterprisedb.com > The Enterprise PostgreSQL Company > > On Mon, Aug 7, 2017 at 8:55 AM, Sarah McAlear <smcal...@pivotal.io> wrote: > >> Hi Harshal! >

Re: [pgAdmin4][patch] update the alert style in the sub-navigation

2017-08-10 Thread Sarah McAlear
ced that due to this patch, the alert style of "Database >>>> connected" message is changed. >>>> Can you please look into this? >>>> >>>> Thanks, >>>> Surinder >>>> >>>> On Wed, Aug 9, 2017 at 4:43

Re: [pgAdmin4][patch] update the alert style in the sub-navigation

2017-08-09 Thread Sarah McAlear
Wed, Aug 9, 2017 at 4:06 PM, Sarah McAlear <smcal...@pivotal.io> wrote: > Hi Surinder! > > I am not able to see anything different from what I see on Master with or > without the patch applied. I tried adjusting the preferences. I did update > the dashboard.js to instantiate

Re: [pgadmin4] unable to pull remote

2017-08-09 Thread Sarah McAlear
generate redirects so it will work, but you both miss out on the security > and will get lower performance because it will be more requests. > > //Magnus > > > On Wed, Aug 9, 2017 at 11:30 AM, Sarah McAlear <smcal...@pivotal.io> > wrote: > >> Great, t

Re: [pgadmin4] unable to pull remote

2017-08-09 Thread Sarah McAlear
it.postgresql.org/git/pgadmin4.git > <http://git.postgresql.org/git/pgadmin4.git>* > > > > -- > *Harshal Dhumal* > *Sr. Software Engineer* > > EnterpriseDB India: http://www.enterprisedb.com > The Enterprise PostgreSQL Company > > On Wed, Aug 9, 201

[pgadmin4] unable to pull remote

2017-08-09 Thread Sarah McAlear
Hi Hackers! We noticed that as of some time today (it worked this morning) we are unable to pull git.postgresql.org/git/pgadmin4.git. Is this a known issue? Our other repos seem to work fine still. Thanks, Sarah

Re: [pgAdmin4][patch] update the alert style in the sub-navigation

2017-08-09 Thread Sarah McAlear
hang <wzh...@pivotal.io> wrote: >>> >>> Hi Ashesh, >>>> >>>> That's correct. This patch just changed alert style in the 'tabs', >>>> such as Dependency and Dependents. >>>> >>>> Thanks >>>>

Re: [pgAdmin4][patch] Fix feature tests failure

2017-08-08 Thread Sarah McAlear
Hi Hackers! Could someone review this patch, please? Thanks so much! Sarah On Mon, Aug 7, 2017 at 4:42 PM, Wenlin Zhang wrote: > Hi Hackers, > > This patch is about fixing the feature tests failure. > > Thanks, > Wenlin, Violet & Hao >

Re: [pgAdmin4][RM2586] Cleanup feature test

2017-08-06 Thread Sarah McAlear
Hi Harshal! There are a few files in which there are leftover imports that are not needed anymore: - PGDataypeFeatureTest - PgadminPage - CheckForXssFeatureTest - CheckDebuggerForXssFeatureTest We also noticed that there were quite a few time.sleep functions that were removed. This

Re: [pgAdmin4][patch] update the alert style in the sub-navigation

2017-08-06 Thread Sarah McAlear
Hi hackers, Could you please review this patch? Thanks Wenlin and Sarah On Wed, Aug 2, 2017 at 2:15 PM, Wenlin Zhang wrote: > Hi Hackers, > > This patch changes the alert style in the sub-navigation to match style > guide. > > Thanks, > Wenlin, Shirley & Sarah > > >

Re: Discussion on improving alertify notifications logic

2017-07-27 Thread Sarah McAlear
Hi Akshay! That seems like a good idea. When we're using the extended pgadmin.alerrtify in the codebase it would be beneficial to call it pgAdminAlertify or something similar so that it is clear that it isn't only the alertify library, but it's been extended. Rob & Sarah On Thu, Jul 27, 2017 at

Re: [pgAdmin4][Patch]: Allow user to Comment/Uncomment code in query editor

2017-07-21 Thread Sarah McAlear
Hi! We looked at the patch and realized that the function to check if an element is not clickable because it's out of view was using an incorrect assertion. We changed it and created a small patch. We also realized that on the current master the yarn.lock is causing changes when running the app.

Re: [pgAdmin4][Patch]: Allow user to Comment/Uncomment code in query editor

2017-07-20 Thread Sarah McAlear
Hello, Attached is a refactor that extracts the keyAction function in the sqlEditor and placed it into a keyboard_shortcuts file. The extracted code is unit tested. I did not look at the feature tests. We are planning on diving into it a bit more but maybe not immediately. Up next we will be

Re: [pgAdmin4][Patch]: Allow user to Comment/Uncomment code in query editor

2017-07-20 Thread Sarah McAlear
Hi Surinder & Murtuza! We are actually working on that right now. We should be sending a patch soon. Thanks! Shruti & Sarah On Thu, Jul 20, 2017 at 7:52 AM, Surinder Kumar < surinder.ku...@enterprisedb.com> wrote: > Hi Murtuza, > > Can the code related to Menu KeyEvents be moved out into a

Re: [PATCH] Persist opened nodes in tree

2017-07-18 Thread Sarah McAlear
Hello, In order to save everything in the backend sqlite database, maybe a jQuery plugin needs to be developed that has the same interface as jStorage and that will do Ajax calls instead of using the Local Storage. The only caveat for this is that it needs to be registered in window. $.jStorage

[pgadmin4][patch] fix history panel on ie11

2017-07-07 Thread Sarah McAlear
Hi Hackers! This patch fixes the layout issues on IE11. There is a known issue on react-split-pane that requires a previous version of bowser in order to render correctly on IE11. We had to downgrade the version of bowser to 1.6.1 until the relevant pull request to react-split-pane has been

[pgadmin-hackers][pgAdmin4][Patch] Make karma test results output to browser

2017-07-06 Thread Sarah McAlear
Hi Hackers! Attached is a minor patch that enables pretty html output of our karma test results. While running yarn test:karma, visit localhost:9876/debug.html to see test output. Thanks, Matt and Sarah diff --git a/web/karma.conf.js b/web/karma.conf.js index 39697ca7..e9b0e062 100644 ---

Re: [pgadmin-hackers] [pgAdmin4] [PATCH] History Tab rewrite in React

2017-06-21 Thread Sarah McAlear
Hello! Thank you for committing the patch! We are currently looking into CEF ( https://bitbucket.org/chromiumembedded/cef) as an alternative to QTWebKit. So far it looks promising. It works on all platforms. It is the base for Google Chrome, so it should be maintained for some time to come.