public inbox for [email protected]
help / color / mirror / Atom feedFrom: Aditya Toshniwal <[email protected]>
To: Dave Page <[email protected]>
Cc: Nikhil Mohite <[email protected]>
Cc: pgadmin-hackers <[email protected]>
Subject: Re: Regarding removal for wcDocker and replace it with rc-dock #6479
Date: Fri, 11 Aug 2023 14:17:27 +0530
Message-ID: <CAM9w-_kvVZfjAPqfmaK3niUcpeE=SuZ_F8BW1WS9g8zCw-t6ew@mail.gmail.com> (raw)
In-Reply-To: <CA+OCxozkx5iuKNzS9MEEUaBnWTG-e5AeU0gcg87gv+pqvBUxtA@mail.gmail.com>
References: <CAM9w-_muqgKY=se3NLgkHpWbOxMkLP9LkB2EMF2VoAkE0ffaJw@mail.gmail.com>
<CAOBg0AOe6mOzHnThmy_YQj73+3bQzcmffsDOOun+nrk3fdfNRg@mail.gmail.com>
<CAM9w-_nj1MaC9jdzoOWrsQroCNpL8BOeAG_7t3XjBSaHjbZeVg@mail.gmail.com>
<CA+OCxozkx5iuKNzS9MEEUaBnWTG-e5AeU0gcg87gv+pqvBUxtA@mail.gmail.com>
Hi Dave,
On Fri, Aug 11, 2023 at 2:05 PM Dave Page <[email protected]> wrote:
>
>
> On Fri, 11 Aug 2023 at 07:42, Aditya Toshniwal <
> [email protected]> wrote:
>
>> Hi Nikhil,
>>
>> On Fri, Aug 11, 2023 at 11:06 AM Nikhil Mohite <
>> [email protected]> wrote:
>>
>>> Hi Aditya,
>>>
>>> On Fri, Aug 11, 2023 at 12:56 AM Aditya Toshniwal <
>>> [email protected]> wrote:
>>>
>>>> Hi Hackers,
>>>>
>>>> I am working on removing the jQuery based wcDocker with ReactJS based
>>>> rc-dock.
>>>> The new lib allows us to reset layout without reloading pgAdmin. The
>>>> one thing it doesn't provide is adding a panel using tab bar context menu.
>>>> I was able to add a custom context menu but only on the tabs and not on the
>>>> complete tab bar.
>>>> So my question is, do we need something to add a panel back or reset
>>>> layout should be enough as we'll not reload pgAdmin?
>>>> If yes, please suggest on how we can do it without using the context
>>>> menu. One way is to have a menu with check menu items for each default
>>>> panel.
>>>> Attached is a screen recording to show the current behavior.
>>>>
>>>> If we add a button in browser explorer(We can change the location of a
>>> button) and click on it will show a drop-down menu with a list of all
>>> closed tabs, and selecting it will open the selected tab on the right side.
>>> Something like this, click on the (+) plus button on Object Explorer and
>>> select the tab name from drop-down options. (I know there are Query Tool,
>>> Search, and PSQL Tool buttons already there we can change the location of
>>> the button)
>>>
>> I think we can do that, but on the main panel and not on object explorer
>> and with some relevant icon.
>> Any other suggestions/ideas?
>>
>
> I'm surprised a man of your talents can't catch the right-click - any
> particular reason it's giving you problems?
>
Thank you for your appreciation. The problem here is rc-dock doesn't
provide any API to allow the onContextMenu event on the tab bar. I could
hack and add using JS native querySelector and addListener, but that is
error prone and not the right way to do it in React. I have used a custom
component for tab title, so I was able to capture the event on the tabs.
>
> Assuming it really is that hard, can we do something like Chrome (similar
> to Nikhil's suggestion)?
>
> [image: Screenshot 2023-08-11 at 09.33.53.png]
>
Chrome has only one type of tab, but pgAdmin has many. I am thinking of
something similar to PyCharm.
[image: image.png]
>
>
>> [image: image.png]
>>>
>>>>
>>>>
>>>> --
>>>> Thanks,
>>>> Aditya Toshniwal
>>>> pgAdmin Hacker | Sr. Software Architect | *enterprisedb.com*
>>>> <https://www.enterprisedb.com/;
>>>> "Don't Complain about Heat, Plant a TREE"
>>>>
>>>
>>> Thanks,
>>> Nikhil Mohite
>>>
>>
>>
>> --
>> Thanks,
>> Aditya Toshniwal
>> pgAdmin Hacker | Sr. Software Architect | *enterprisedb.com*
>> <https://www.enterprisedb.com/;
>> "Don't Complain about Heat, Plant a TREE"
>>
>
>
> --
> Dave Page
> Blog: https://pgsnake.blogspot.com
> Twitter: @pgsnake
>
> EDB: https://www.enterprisedb.com
>
>
--
Thanks,
Aditya Toshniwal
pgAdmin Hacker | Sr. Software Architect | *enterprisedb.com*
<https://www.enterprisedb.com/;
"Don't Complain about Heat, Plant a TREE"
Attachments:
[image/png] image.png (449.0K, 3-image.png)
download | view image
[image/png] Screenshot 2023-08-11 at 09.33.53.png (16.9K, 4-Screenshot%202023-08-11%20at%2009.33.53.png)
download | view image
[image/png] image.png (128.1K, 5-image.png)
download | view image
view thread (7+ messages) latest in thread
reply
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Reply to all the recipients using the --to and --cc options:
reply via email
To: [email protected]
Cc: [email protected], [email protected], [email protected]
Subject: Re: Regarding removal for wcDocker and replace it with rc-dock #6479
In-Reply-To: <CAM9w-_kvVZfjAPqfmaK3niUcpeE=SuZ_F8BW1WS9g8zCw-t6ew@mail.gmail.com>
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
This inbox is served by agora; see mirroring instructions
for how to clone and mirror all data and code used for this inbox