Re: [pgadmin-hackers][pgaweb][patch] Website patch to add SCSS

2017-07-12 Thread George Gelashvili
Hi Dave, > Hmm, seems like the dependencies are broken. Adding them throws me down > what seems like a rabbit hole of manual installation, starting with: > > (pgaweb) piranha:pgaweb dpage$ pip install -r requirements.txt > Requirement already satisfied: Django==1.8.18 in /Users/dpage/.virtualenvs

make subnode rows the same height for rows without dropdowns

2017-07-05 Thread George Gelashvili
a fix. Thanks, Shirley and Shruti and George commit 451ca8d14ad6fe19871dd9325a19d2a25c540422 Author: George Gelashvili, Shirley Wang and Shruti Iyer Date: Wed Jul 5 12:25:15 2017 -0400 make subnode rows the same height for rows without dropdowns diff --git a/web/pgadmin/stat

Re: [pgAdmin4][Patch]: Refactor of the History Tab

2017-07-03 Thread George Gelashvili
Hi > 2) Can 1px top border be added above first entry in left panel to > differentiate from above panels ? > > It sounds like you're describing the distinction between the query history entries and the Messages, History, etc. tab bar (screenshot attached). If so, the gray/blue contrast should be s

Re: [pgAdmin4][Patch] To fix the duplicate entry in History panel

2017-06-30 Thread George Gelashvili
Nice! Looks good. We were investigating this bug before you sent this and it looks like your patch resolves the duplicate calls to update_msg_history Cheers, Sarah and George On Fri, Jun 30, 2017 at 1:14 PM, Murtuza Zabuawala < murtuza.zabuaw...@enterprisedb.com> wrote: > Hi, > > PFA minor patch

Re: [pgAdmin4]: Webpacking of static JS/CSS

2017-06-29 Thread George Gelashvili
Hey Surinder, ​1. ​ > Tools > ​(tools.js)​ - It will contain all JS modules under tools directory > > > ​2. ​ > Browser > ​(browser.js)​ - It will contain all JS modules under browser directory > By under, do you mean every javascript file recursively under the browser directory? > ​4. Common