Re: [pgAdmin4][RM3480] User can not see any SQL query in SQL pane

From: Dave Page <dpage(at)pgadmin(dot)org>
To: Aditya Toshniwal <aditya(dot)toshniwal(at)enterprisedb(dot)com>
Cc: pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org>
Subject: Re: [pgAdmin4][RM3480] User can not see any SQL query in SQL pane
Date: 2018-07-10 10:07:50
Message-ID: CA+OCxozbPsWe8OEjA=TihE0dwFoKRXVwnkKKUHkz5U_Qv2wf1A@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgadmin-hackers

Hi

On Tue, Jul 10, 2018 at 10:36 AM, Aditya Toshniwal <
aditya(dot)toshniwal(at)enterprisedb(dot)com> wrote:

> Hi Hackers,
>
> Attached is the fix. It is again related to changed made for RM3294, the
> font size was set to 0em. Also included changes to make sure CodeMirror
> font size is never set to 0em but to default 1em if font size is not
> available from preferences.
>

Unfortunately this fails linting, and when that is fixed, fails the tests:

HeadlessChrome 0.0.0 (Mac OS X 10.12.6) SqlEditorUtils Calculate font size
of input number passed calcFontSize FAILED

Expected '1em' to equal '0em'.

at UserContext.<anonymous>
(regression/javascript/webpack:/regression/javascript/sqleditor_utils_spec.js:35:47)

HeadlessChrome 0.0.0 (Mac OS X 10.12.6): Executed 660 of 741 (1 FAILED) (0
secs / 7.075 secs)

HeadlessChrome 0.0.0 (Mac OS X 10.12.6) SqlEditorUtils Calculate font size
of input number passed calcFontSize FAILED

Expected '1em' to equal '0em'.

HeadlessChrome 0.0.0 (Mac OS X 10.12.6): Executed 741 of 741 (1 FAILED)
(7.74 secs / 7.383 secs)

error Command failed with exit code 1.

info Visit *https://yarnpkg.com/en/docs/cli/run
<https://yarnpkg.com/en/docs/cli/run>* for documentation about this command.

make: *** [check-js] Error 1

--
Dave Page
Blog: http://pgsnake.blogspot.com
Twitter: @pgsnake

EnterpriseDB UK: http://www.enterprisedb.com
The Enterprise PostgreSQL Company

In response to

Responses

Browse pgadmin-hackers by date

  From Date Subject
Next Message Akshay Joshi 2018-07-10 10:20:17 Regarding RM #3316 Pgadmin4 No Tray Crash
Previous Message Dave Page 2018-07-10 10:00:09 Re: [pgAdmin4][patch] jQuery 3 migration few more leftovers