From: | Ashesh Vashi <ashesh(dot)vashi(at)enterprisedb(dot)com> |
---|---|
To: | Sarah McAlear <smcalear(at)pivotal(dot)io> |
Cc: | pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org>, Bing Xu <bxu(at)pivotal(dot)io> |
Subject: | Re: Code Folding in Code Mirror |
Date: | 2017-08-16 03:54:02 |
Message-ID: | CAG7mmoyGfaMzFx0UOxdkpvqLpM1v1D7wVS52E=90TVerYPJ+7A@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgadmin-hackers |
On Tue, Aug 15, 2017 at 2:33 PM, Sarah McAlear <smcalear(at)pivotal(dot)io> wrote:
> Ok, it does work for the cases defined. We misunderstood how it works.
>
Still, could we enable this everywhere?
>
Yes.
> What is the reason not to?
>
I don't see any reason not to do it.
-- Thanks, Ashesh
>
> Thanks,
> Bing & Sarah
>
> On Tue, Aug 15, 2017 at 5:00 PM, Sarah McAlear <smcalear(at)pivotal(dot)io>
> wrote:
>
>> Hi Hackers!
>>
>> We are working on unifying the CodeMirror stuff into one location. While
>> looking at this we noticed that while there is logic for code folding, it
>> doesn't seem to be working. Does it work for everyone else? We also started
>> to wonder if there is a specific reason that code folding should be
>> disabled anywhere? Would it not make more sense to have it look the same
>> throughout pgAdmin? We understand that there are a few things that should
>> be depending on the place in the app where it is used, such as 'editable'.
>> Other than that, would it be acceptable to have the same configuration
>> throughout?
>>
>> Thanks,
>> Bing & Sarah
>>
>
>
From | Date | Subject | |
---|---|---|---|
Next Message | Surinder Kumar | 2017-08-16 05:34:28 | Re: [pgAdmin4][patch] update the alert style in the sub-navigation |
Previous Message | Violet Cheng | 2017-08-16 03:29:42 | Re: [pgadmin4][Patch] Greenplum specific DDL and Dashboard display |