From: | Yosry Muhammad <yosrym93(at)gmail(dot)com> |
---|---|
To: | Dave Page <dpage(at)pgadmin(dot)org> |
Cc: | Aditya Toshniwal <aditya(dot)toshniwal(at)enterprisedb(dot)com>, pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org> |
Subject: | Re: [GSoC] Query History Integration with updatable query resultsets and improvements |
Date: | 2019-08-07 08:35:08 |
Message-ID: | CAFSMqn9jB04Q3AFg_XEfbnK-fk+c9ag0_=AMoqBJz=E=mG5nLg@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgadmin-hackers |
Hi,
On Wed, Aug 7, 2019, 10:31 AM Dave Page <dpage(at)pgadmin(dot)org> wrote:
> Hi
>
> On Tue, Aug 6, 2019 at 5:46 PM Yosry Muhammad <yosrym93(at)gmail(dot)com> wrote:
>
>> Hi,
>>
>> On Tue, Aug 6, 2019, 6:01 PM Dave Page <dpage(at)pgadmin(dot)org> wrote:
>>
>>> Hi
>>>
>>> Seems to work well :-). A few comments:
>>>
>>> - Should we have an icon by the non-generated queries (the lightning
>>> flash) and the COMMITs etc (the commit/rollback icon as appropriate) as
>>> well? I think just having the icon for generated queries looks a little odd.
>>>
>>
>> I am sorry I don't quite understand. Do you mean having an icon to
>> show/hide user queries as well? I believe a checkbox is more appropriate
>> anyway, maybe placed right above the history entries. I am going to need a
>> design of the checkbox or styling guidlines though.
>>
>
> No, I mean put an icon by all queries to indicate the source of them. We'd
> probably want one for the EXPLAIN button too.
>
Okay, got it.
> For showing/hiding generate queries, we should use one of our slider
> switches that we use elsewhere.
>
> One other thing I found: The Copy to Query Editor button has gone missing,
> and the Copy button seems to have stopped working.
>
>
>>
I will check that, thanks for the feedback.
From | Date | Subject | |
---|---|---|---|
Next Message | Aditya Toshniwal | 2019-08-07 09:05:42 | Re: [GSoC] Query History Integration with updatable query resultsets and improvements |
Previous Message | Dave Page | 2019-08-07 08:31:02 | Re: [GSoC] Query History Integration with updatable query resultsets and improvements |