Re: RM2233 - Scratch pad

Hi Dave

Following are my review comments:

- When query is running then user should be able to work on Scratch pad.
- Try to resize the Data Output panel or just click multiple times on
"Query Editor" Tab Scratch pad 's width will increased on every click.
- By default Scratch pad tab should be docked at right side. Currently
it depends on your mouse click. On mouse click it shows the area where it
will be docked. Refer attached Scratch_Pad_Area.png and Scratch_Pad_Dock.
png
- Got error in browser when hover on Add Panel menu. Refer attached
Hover_Error.png
- Do we need Scratch pad in View/Edit Data?

On Fri, Feb 15, 2019 at 6:59 PM Khushboo Vashi <
khushboo(dot)vashi(at)enterprisedb(dot)com> wrote:

>
>
> On Fri, Feb 15, 2019 at 6:35 PM Dave Page <dpage(at)pgadmin(dot)org> wrote:
>
>> Hi
>>
>> On Fri, Feb 15, 2019 at 11:53 AM Khushboo Vashi
>> <khushboo(dot)vashi(at)enterprisedb(dot)com> wrote:
>> >
>> > Hi Dave,
>> >
>> > On Thu, Feb 14, 2019 at 8:14 PM Dave Page <dpage(at)pgadmin(dot)org> wrote:
>> >>
>> >> Here's a draft patch for adding a scratch pad to the Query Tool,
>> >> similar to what we had in pgAdmin 3.
>> >>
>> >> Aside from regular review, can one of our CSS gurus (I'm looking at
>> >> you Aditya and Khushboo :-) ) help with the following questions:
>> >>
>> >> 1) I'm currently sizing the text area height using "height: calc(100%
>> >> - 3px);" to avoid a vertical scrollbar in one of the parent divs
>> >> created by wcDocker. I'm sure there's a more elegant way of doing that
>> >> (and one that's not likely to be so fragile).
>> >>
>> > Can't we just add height: 100% instead of calc?
>>
>> No - that causes the scrollbar to appear.
>>
>> > One more way to avoid vertical scroll-bar is overflow-y: hidden
>>
>> wouldn't that need to go in the div that wcDocker is creating?
>
> We can apply it on the parent-div which is div.sql-scratch
>
>>
>> >> 2) I've enabled the wcDocker context menu so users can show or hide
>> >> the scratch pad. The sub-menu is expanding as shown in the attached
>> >> screenshot though, however, it works fine on the main browser wcDocker
>> >> instance.
>> >>
>> > Please add 'jquery.ui', 'jqueryui.position' dependencies in
>> sqleditor.js .
>>
>> Thanks, I'll give it a try.
>>
>>
>> --
>> Dave Page
>> Blog: http://pgsnake.blogspot.com
>> Twitter: @pgsnake
>>
>> EnterpriseDB UK: http://www.enterprisedb.com
>> The Enterprise PostgreSQL Company
>>
>

--
*Akshay Joshi*

*Sr. Software Architect *

*Phone: +91 20-3058-9517Mobile: +91 976-788-8246*