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

From: Aditya Toshniwal <aditya(dot)toshniwal(at)enterprisedb(dot)com>
To: Dave Page <dpage(at)pgadmin(dot)org>
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:58:04
Message-ID: CAM9w-_mNR0UpMv4mnm=NDfttDr+kfTPvDs7nJy9gcqMQEmi1iQ@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgadmin-hackers

Hi,

On Tue, Jul 10, 2018 at 4:24 PM, Dave Page <dpage(at)pgadmin(dot)org> wrote:

> I think you forgot the patch :-/
>
​Apologies. I know this is annoying :(

PFA.

>
>
> On Tue, Jul 10, 2018 at 11:48 AM, Aditya Toshniwal <aditya.toshniwal@
> enterprisedb.com> wrote:
>
>> Hi Hackers,
>>
>> On Tue, Jul 10, 2018 at 3:37 PM, Dave Page <dpage(at)pgadmin(dot)org> wrote:
>>
>>> 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:
>>>
>> ​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
>>> <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"
>>
>
>
>
> --
> 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"

Attachment Content-Type Size
RM3480.patch application/octet-stream 2.6 KB

In response to

Responses

Browse pgadmin-hackers by date

  From Date Subject
Next Message Dave Page 2018-07-10 12:58:48 pgAdmin 4 commit: Ensure CodeMirror always has a non-zero font size. Fi
Previous Message Dave Page 2018-07-10 10:54:01 Re: [pgAdmin4][RM3480] User can not see any SQL query in SQL pane