Received: from malur.postgresql.org ([217.196.149.56]) by arkaria.postgresql.org with esmtps (TLS1.3) tls TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384 (Exim 4.94.2) (envelope-from ) id 1rx7ci-006V5k-HN for pgadmin-hackers@arkaria.postgresql.org; Wed, 17 Apr 2024 15:55:00 +0000 Received: from localhost ([127.0.0.1] helo=malur.postgresql.org) by malur.postgresql.org with esmtp (Exim 4.94.2) (envelope-from ) id 1rx7cg-00CcpF-Hi for pgadmin-hackers@arkaria.postgresql.org; Wed, 17 Apr 2024 15:54:58 +0000 Received: from makus.postgresql.org ([2001:4800:3e1:1::229]) by malur.postgresql.org with esmtps (TLS1.3) tls TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384 (Exim 4.94.2) (envelope-from ) id 1rx7cg-00Ccot-5E for pgadmin-hackers@lists.postgresql.org; Wed, 17 Apr 2024 15:54:58 +0000 Received: from mail-oa1-x2e.google.com ([2001:4860:4864:20::2e]) by makus.postgresql.org with esmtps (TLS1.3) tls TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256 (Exim 4.94.2) (envelope-from ) id 1rx7cY-003JYt-HO for pgadmin-hackers@postgresql.org; Wed, 17 Apr 2024 15:54:56 +0000 Received: by mail-oa1-x2e.google.com with SMTP id 586e51a60fabf-232f2b86e4fso2915817fac.3 for ; Wed, 17 Apr 2024 08:54:50 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=enterprisedb.com; s=google; t=1713369289; x=1713974089; darn=postgresql.org; h=cc:to:subject:message-id:date:from:in-reply-to:references :mime-version:from:to:cc:subject:date:message-id:reply-to; bh=0GdCBne6DrZ4REq/cWHtqxf458kdV7jF+xEVa7PKWSs=; b=GoDoGdmdxiUj3NuAmdlkkHV/c88bw0xqHhdeYpah8Chi/eS59khT0+oIiWzYrRwVZn Chx82g/ZP1nAU/gQkaZNzk6nyd69XXWoY6NbBc9iHFuq1FyU4OD/7voJYkknQ2WKYAbD /YQFZ18qB+rgqATp+Tzl/T2Bnd7PLks0CGXpUs3giQhkbPKMCIoF6PU3NJU6qlI0eCvW qm4zuI+ZUw44PYMQ0uAGHVwtVK4g5CddREUJV4DpYRLZ/jh1cVDux2bSZgRaaXo2ywtI N38tVIro7xJhWX+/bFXgW0ZjaxRg+g3k49dnBgg/mRyqT8RuNE8a+fHL2U2kF0l7G32U kTtw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1713369289; x=1713974089; h=cc:to:subject:message-id:date:from:in-reply-to:references :mime-version:x-gm-message-state:from:to:cc:subject:date:message-id :reply-to; bh=0GdCBne6DrZ4REq/cWHtqxf458kdV7jF+xEVa7PKWSs=; b=EYeORADNsEcvMEvj7iXjuGEHz2uSlzohyhD11KNzf5pu8NzKV00qyLqtx/5l2GSGSZ rsUpLEcedHwbIZ18jAIbWYQo0zpCCK2absP1x6X+xysaIjMTUfMP1qR6KFoaiPw9P6KN pCa3zRw8kcqQ7hHmd+aVpnkObTyugAWx3Kcn4Bl+oLQZ8I/+LO2qPO4x3M4Z2EqX8GQA NlF7vXXduICpP5FeCZNpWuXIvBUtzS13g80pN7NtLqJHIdvGzV1kY4gmaHNfI8JqQn1r wPO7VcDDGd1tas00W2p8cxFFhQh/7KrpJ/vjzIOyvoVZK6yGI51Wu1Cx/qnB3FxEYSNH qcSA== X-Gm-Message-State: AOJu0YwM6OnvEDcLW0qLUwrrsuyWD5qCArqIF7zawLGn1ShC5BCsS5qw 1meNkb/KMLSOqwbnxaCqGogXLyldADQ4wvpqmWMKKoR413hi5i0y+GGDku6p4Of3t7FF7YOcecX FDOztKuO6U0tuk9gQ1aRO7lgwQFLK5okX1n4V X-Google-Smtp-Source: AGHT+IHE6VjOHbvOr15aLRBudcxbRlaNLRoxiIQv2/qJaSl4NGVqLWBTSjSFXyv12bB4h2NkfHf6kX7rrenldnwbOY4= X-Received: by 2002:a05:6870:b294:b0:233:5a50:ba3c with SMTP id c20-20020a056870b29400b002335a50ba3cmr17207303oao.19.1713369289560; Wed, 17 Apr 2024 08:54:49 -0700 (PDT) MIME-Version: 1.0 References: In-Reply-To: From: Aditya Toshniwal Date: Wed, 17 Apr 2024 21:24:13 +0530 Message-ID: Subject: Re: Regarding feature #6841 To: Anil Sahoo Cc: pgadmin-hackers@postgresql.org Content-Type: multipart/alternative; boundary="000000000000dfb75906164cdf1f" List-Id: List-Help: List-Subscribe: List-Post: List-Owner: List-Archive: Archived-At: Precedence: bulk --000000000000dfb75906164cdf1f Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable Hi Anil, I think the toolbar query should look like an input box. And the tooltip with a codemirror is more UI friendly. On Wed, Apr 17, 2024 at 7:38=E2=80=AFPM Anil Sahoo wrote: > Hi Hackers, > > This feature is about executing a query at the cursor position. And that > query can be a one line or multiline. I have assigned a play icon button > and F5 as the keyboard shortcut for the Execute Query feature, and for > Execute Script, Playlist icon button and Alt+F5(Others),Option+F5(Mac) as > keyboard shortcut. > > As now the query can run at cursor position, so for user convenience I am > showing the current query just beside the Data Output toolbar. And on hov= er > of the text, it will show the whole query as a tooltip. This query text > will be available for both Execute Script and Execute Query. > > I have made the UI change for the feature #6841 > . > > Please provide your suggestions and feedback if these changes look okay t= o > you. > > Regards, > Anil > -- > > > > *Anil Sahoo* > > Software Engineer > > www.enterprisedb.com > > Power to Postgres > > > > > > --=20 Thanks, Aditya Toshniwal pgAdmin Hacker | Sr. Software Architect | *enterprisedb.com* "Don't Complain about Heat, Plant a TREE" --000000000000dfb75906164cdf1f Content-Type: text/html; charset="UTF-8" Content-Transfer-Encoding: quoted-printable
Hi=C2=A0Anil,

I think the toolbar query should look like a= n input box. And the tooltip with a codemirror is more UI friendly.=C2=A0

On Wed, Apr 17, 2024 at 7:38=E2=80=AFPM Anil Sahoo <anil.sahoo@enterprisedb.com> wrote:
= Hi Hackers,

This feature is about executing a query=C2= =A0at the=C2=A0cursor position. And that query can be a one=C2=A0line or mu= ltiline. I have assigned a play icon button and F5 as the keyboard shortcut= for the Execute Query feature, and for Execute Script, Playlist icon butto= n and Alt+F5(Others),Option+F5(Mac) as keyboard shortcut.

As now the query can run at cursor position, so for user convenienc= e I am showing the current query just beside the Data=C2=A0Output toolbar. = And on hover of the text, it will show the whole query as a tooltip. This q= uery text will be available for both Execute Script and Execute Query.

I have made the UI change for the feature #6841= .=C2=A0

Please provide your suggestions and fe= edback if these changes look okay to you.

Regards,=
Anil
--


--
Thanks,
Aditya Toshniwal
pgAdmin Hacker=C2=A0| Sr. Software Architect=C2=A0| enterprisedb.com

=

Anil Sahoo

Softwar= e Engineer

www.enterprisedb.com

Power to Postgres

=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0