Working on it. -- Rob
On Thu, Jul 6, 2017 at 10:03 AM, Dave Page <dp...@pgadmin.org> wrote: > And, Houston, we have a problem. The layout is quite broken on IE - which > of course we're working to use on Windows. Can you take a look ASAP please > Matt & friends? > > Screenshot attached. > > On Thu, Jul 6, 2017 at 1:08 PM, Dave Page <dp...@pgadmin.org> wrote: > >> Thanks, applied. >> >> On Wed, Jul 5, 2017 at 4:31 PM, Matthew Kleiman <mklei...@pivotal.io> >> wrote: >> >>> Sorry Dave. It looks like we had formatted the patch incorrectly. We >>> have recreated the patch. >>> >>> Matt and Sarah >>> >>> On Wed, Jul 5, 2017 at 10:49 AM, Dave Page <dp...@pgadmin.org> wrote: >>> >>>> Hi >>>> >>>> On Wed, Jul 5, 2017 at 2:54 PM, Matthew Kleiman <mklei...@pivotal.io> >>>> wrote: >>>> >>>>> 1) I see the font used for class .query-history .entry is monospace, >>>>>> shouldn't it be Helvetica as per style guide ? the font for Messages > >>>>>> text >>>>>> is also monospace. >>>>>> >>>>> >>>>> As per Dave's comment, we have added this to the styleguide backlog. >>>>> We will be adding monospace entries to the styleguide. >>>>> >>>>> 2) Can 1px top border be added above first entry in left panel to >>>>>> differentiate from above panels ? >>>>> >>>>> >>>>> We have updated the patch to include a 1px solid #cccccc border at the >>>>> top. >>>>> >>>>> 3) No query execution message appears if i run same query second time, >>>>>> please refer screenshot. >>>>> >>>>> >>>>> Appears to be fixed with Murtuza's patch. >>>>> >>>> >>>> I get the following error with this patch when bundling: >>>> >>>> [235] ./scss/pgadmin.scss 1.1 kB {3} [built] [failed] [1 error] >>>> + 363 hidden modules >>>> >>>> ERROR in ./scss/pgadmin.scss >>>> Module build failed: >>>> @import 'primaryblue'; >>>> ^ >>>> File to import not found or unreadable: primaryblue. >>>> Parent style sheet: stdin >>>> in /Users/dpage/git/pgadmin4/web/pgadmin/static/scss/pgadmin.scss >>>> (line 4, column 1) >>>> >>>> ERROR in ./scss/pgadmin.scss >>>> Module build failed: ModuleBuildError: Module build failed: >>>> @import 'primaryblue'; >>>> ^ >>>> File to import not found or unreadable: primaryblue. >>>> Parent style sheet: stdin >>>> in /Users/dpage/git/pgadmin4/web/pgadmin/static/scss/pgadmin.scss >>>> (line 4, column 1) >>>> at runLoaders (/Users/dpage/git/pgadmin4/web >>>> /node_modules/webpack/lib/NormalModule.js:192:19) >>>> at /Users/dpage/git/pgadmin4/web/node_modules/loader-runner/lib >>>> /LoaderRunner.js:364:11 >>>> at /Users/dpage/git/pgadmin4/web/node_modules/loader-runner/lib >>>> /LoaderRunner.js:230:18 >>>> at context.callback (/Users/dpage/git/pgadmin4/web >>>> /node_modules/loader-runner/lib/LoaderRunner.js:111:13) >>>> at Object.asyncSassJobQueue.push [as callback] >>>> (/Users/dpage/git/pgadmin4/web/node_modules/sass-loader/lib/ >>>> loader.js:55:13) >>>> at Object.<anonymous> (/Users/dpage/git/pgadmin4/web >>>> /node_modules/async/dist/async.js:2243:31) >>>> at Object.callback (/Users/dpage/git/pgadmin4/web >>>> /node_modules/async/dist/async.js:906:16) >>>> at options.error (/Users/dpage/git/pgadmin4/web >>>> /node_modules/node-sass/lib/index.js:294:32) >>>> Child extract-text-webpack-plugin: >>>> [0] /Users/dpage/git/pgadmin4/web/~/css-loader!/Users/dpage/git/ >>>> pgadmin4/web/~/sass-loader/lib/loader.js!./scss/pgadmin.scss 247 bytes >>>> {0} [built] [failed] [1 error] >>>> >>>> ERROR in /Users/dpage/git/pgadmin4/web/ >>>> ~/css-loader!/Users/dpage/git/pgadmin4/web/~/sass-loader/lib >>>> /loader.js!./scss/pgadmin.scss >>>> Module build failed: >>>> @import 'primaryblue'; >>>> ^ >>>> File to import not found or unreadable: primaryblue. >>>> Parent style sheet: stdin >>>> in /Users/dpage/git/pgadmin4/web/pgadmin/static/scss/ >>>> pgadmin.scss (line 4, column 1) >>>> >>>> >>>> >>>> -- >>>> 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 >> > > > > -- > Dave Page > Blog: http://pgsnake.blogspot.com > Twitter: @pgsnake > > EnterpriseDB UK: http://www.enterprisedb.com > The Enterprise PostgreSQL Company >