Hi, On Tue, Jul 10, 2018 at 4:24 PM, Dave Page <dp...@pgadmin.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 <dp...@pgadmin.org> wrote: >> >>> Hi >>> >>> On Tue, Jul 10, 2018 at 10:36 AM, Aditya Toshniwal < >>> aditya.toshni...@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 >>> <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"
RM3480.patch
Description: Binary data