Re: [pgAdmin4][RM3480] User can not see any SQL query in SQL pane - Mailing list pgadmin-hackers

From Aditya Toshniwal
Subject Re: [pgAdmin4][RM3480] User can not see any SQL query in SQL pane
Date
Msg-id CAM9w-_=deejZJeZkyTbbQUmHvfY57aWBcxQAoPdGXx=-FsSx9g@mail.gmail.com
Whole thread Raw
In response to Re: [pgAdmin4][RM3480] User can not see any SQL query in SQL pane  (Dave Page <dpage@pgadmin.org>)
Responses Re: [pgAdmin4][RM3480] User can not see any SQL query in SQL pane  (Dave Page <dpage@pgadmin.org>)
List pgadmin-hackers
Hi Hackers,

On Tue, Jul 10, 2018 at 3:37 PM, Dave Page <dpage@pgadmin.org> wrote:
Hi

On Tue, Jul 10, 2018 at 10:36 AM, Aditya Toshniwal <aditya.toshniwal@enterprisedb.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:
​Not good. Should have checked. Attached is the updated patch.​
 

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 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



--
Thanks and Regards,
Aditya Toshniwal
Software Engineer | EnterpriseDB Software Solutions | Pune
"Don't Complain about Heat, Plant a tree"

pgadmin-hackers by date:

Previous
From: Akshay Joshi
Date:
Subject: Regarding RM #3316 Pgadmin4 No Tray Crash
Next
From: Dave Page
Date:
Subject: Re: [pgAdmin4][RM3480] User can not see any SQL query in SQL pane