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

Reply via email to