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