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 12:58:54 |
Message-ID: | CA+OCxowReUsjsWSe8KWAbNfm6s3G1XagJatqxid4Bd_M-8m3Tw@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgadmin-hackers |
Thanks - patch applied!
On Tue, Jul 10, 2018 at 11:58 AM, Aditya Toshniwal <
aditya(dot)toshniwal(at)enterprisedb(dot)com> wrote:
> 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(dot)toshniwal(at)enterprisedb(dot)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"
>
--
Dave Page
Blog: http://pgsnake.blogspot.com
Twitter: @pgsnake
EnterpriseDB UK: http://www.enterprisedb.com
The Enterprise PostgreSQL Company
From | Date | Subject | |
---|---|---|---|
Next Message | Dave Page | 2018-07-10 13:02:15 | Re: Regarding RM #3316 Pgadmin4 No Tray Crash |
Previous Message | Dave Page | 2018-07-10 12:58:48 | pgAdmin 4 commit: Ensure CodeMirror always has a non-zero font size. Fi |