Received: from malur.postgresql.org ([217.196.149.56]) by arkaria.postgresql.org with esmtp (Exim 4.84_2) (envelope-from ) id 1bnK2a-0006VF-H3 for pgadmin-hackers@arkaria.postgresql.org; Fri, 23 Sep 2016 06:32:56 +0000 Received: from localhost ([127.0.0.1] helo=postgresql.org) by malur.postgresql.org with smtp (Exim 4.84_2) (envelope-from ) id 1bnK2Z-0002PH-IV for pgadmin-hackers@arkaria.postgresql.org; Fri, 23 Sep 2016 06:32:55 +0000 Received: from magus.postgresql.org ([2a02:c0:301:0:ffff::29]) by malur.postgresql.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_CBC_SHA384:256) (Exim 4.84_2) (envelope-from ) id 1bnK2L-0002Ae-Sb for pgadmin-hackers@postgresql.org; Fri, 23 Sep 2016 06:32:41 +0000 Received: from mail-wm0-x234.google.com ([2a00:1450:400c:c09::234]) by magus.postgresql.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_CBC_SHA1:256) (Exim 4.84_2) (envelope-from ) id 1bnK2H-00077i-Vp for pgadmin-hackers@postgresql.org; Fri, 23 Sep 2016 06:32:41 +0000 Received: by mail-wm0-x234.google.com with SMTP id 197so2741588wmk.1 for ; Thu, 22 Sep 2016 23:32:37 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=enterprisedb-com.20150623.gappssmtp.com; s=20150623; h=mime-version:in-reply-to:references:from:date:message-id:subject:to :cc; bh=c3taYonBebsnnPprLRVIGbWW+uNP1FIGSASNagM48bA=; b=vwB5GU0gCkcqoqWRDbsaPdVHM0+ryUuGSVKWkZVv4kHFs4W0iUWnDEa2j6FGR8JWQG Dm80nZrpnGJ2rEjJi8qpWKKzvHeQvmi0ykzkmuJmDnvkhwFuiamnm+QQs4XFK8TpZqNm maHq3gX7NLV3GfyalyQnYVCpqBQ+fCN3MIDX/quMb5YB/iOQ3WaWdj1mRc4LeGRILTWW mZ7iaVYQ88vid9KkMY+ZLnk7hA4SeNGGibFvKIZ7n1N/RAuvheHUwN1TyguogZwyRWoI WqZJ8j84LpQiM44fmBoxslKT+upLa11LYt3epFf4A14HT2wq+hP2etCot3sDdTSean0l SMXA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20130820; h=x-gm-message-state:mime-version:in-reply-to:references:from:date :message-id:subject:to:cc; bh=c3taYonBebsnnPprLRVIGbWW+uNP1FIGSASNagM48bA=; b=COVYrVzalnVYU27an/eXIHdLHuoGn7Xp3t9guZ5wwBflcwx3ADAdwYxOqugnxOJWTv hzCTZ09aDd03aCCdQ0gzNv/zs3M1sCpOz9owdoSD+bJKSomXs0UAcZOr3UyTWWjtZPby MKQtmlToBpRud/fgTg6MA4wEnd0dnFn8Kt/pBouP+JvLBJbqRODZ5v/1aBKqauLUx6gK qaY09L2U160+t8piiUTOXAIV1nBzV5l3JnMc9LUk+88kgb1bxhlkF+ld1xY6aypHZzKk m1X6UZg9GaEGuEWRU+hFLCUkLfwN9f86sseRnXdYgsUKGVnWK4lb1cA6J8FpbuaFySzW rdsA== X-Gm-Message-State: AA6/9RkzbhtNPWtXIqR1dBsEr9st1oeWH5n/gADO1ladtAa3diIKfi8QzXnzoCVPyb4d0+GurhRAYTRVIDn49A2y X-Received: by 10.194.155.100 with SMTP id vv4mr5478095wjb.56.1474612356918; Thu, 22 Sep 2016 23:32:36 -0700 (PDT) MIME-Version: 1.0 Received: by 10.28.172.198 with HTTP; Thu, 22 Sep 2016 23:32:06 -0700 (PDT) In-Reply-To: References: From: Surinder Kumar Date: Fri, 23 Sep 2016 12:02:06 +0530 Message-ID: Subject: Re: Lack of activity indicator over slow connections (pgadmin4) To: Dave Page Cc: Magnus Hagander , pgadmin-hackers , Ashesh Vashi , Murtuza Zabuawala Content-Type: multipart/alternative; boundary=089e01177095b744fb053d26f153 X-Pg-Spam-Score: -2.6 (--) List-Archive: List-Help: List-ID: List-Owner: List-Post: List-Subscribe: List-Unsubscribe: X-Mailing-List: pgadmin-hackers Precedence: bulk Sender: pgadmin-hackers-owner@postgresql.org --089e01177095b744fb053d26f153 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: quoted-printable On Wed, Sep 21, 2016 at 8:40 PM, Dave Page wrote: > Hi > > On Wed, Sep 21, 2016 at 2:30 PM, Surinder Kumar > wrote: > > Hi Dave/Ashesh > > > > The thing we can do is: > > 1) We can display loading message same as other panels if returned > response > > is taking longer than 2 seconds, otherwise we won't display any > message(to > > avoid fluctuations). > > Screenshot - demo_message.png > > There's a typo there (laoded).. > > > 2) Display a loading spinner along with panel title while data is being > > loaded. > > Screenshot - spinner.png > > I don't see a spinner there at all. =E2=80=8Bsorry its not spinner, its reload spinning icon in place of proper= ties icon.=E2=80=8B > > What about the other places I mentioned? > - On dialogue open and OK D=E2=80=8B isplay *loading message* =E2=80=8Bin dialog till the data loads and on OK we can set mouse pointer t= o 'progress' till the data is not processed. =E2=80=8B =E2=80=8B - On Wizard steps =E2=80=8B =E2=80=8B D=E2=80=8B isplay *loading message* =E2=80=8Balso in wizard pages till it doesn't loads data. =E2=80=8B =E2=80=8B - Possibly in the debugger, when stepping? =E2=80=8B =E2=80=8B =E2=80=8B In case of debugger's step into, step out & resume, we can set mouse pointer to 'progress'=E2=80=8B =E2=80=8B. =E2=80=8B > > > On Wed, Sep 21, 2016 at 5:53 PM, Dave Page wrote: > >> > >> Ashesh, Murtuza, Surinder, > >> > >> Some feedback or preferably a patch please? Release is next week! > >> > >> On Fri, Sep 16, 2016 at 3:36 PM, Magnus Hagander > >> wrote: > >> > On Fri, Sep 16, 2016 at 3:35 AM, Dave Page wrote= : > >> >> > >> >> Ashesh, Murtuza, Surinder, > >> >> > >> >> Any thoughts on how we can achieve this easily, and where we're > >> >> lacking notifications? We did have activity notification in the > >> >> treeview, but that never worked overly well and now I come to think > of > >> >> it, I don't recall seeing it recently. I'm thinking we need > something: > >> >> > >> >> - On treeview node click > >> >> > >> >> - On dialogue open and OK > >> >> > >> >> - On Wizard steps > >> >> > >> >> - Possibly in the debugger, when stepping? > >> >> > >> >> I would think all that we need is to set the mouse cursor to > >> >> 'progress'. > >> >> > >> > > >> > Actually, the treeview spinner worked most of them time. I'm not su= re > >> > if it > >> > worked all the time - I didn't check that carefully - but it did wor= k > >> > most > >> > of the mtime. > >> > > >> > But basically whatever *else* I clicked was the problem. In particul= ar > >> > clicking the tabs out right, with the "Properties" and "SQL" tabs > being > >> > the > >> > ones I used and they were extremely slow over the high latency > >> > connection (I > >> > measured almost 4 minutes as the worst time to load a SQL pane for a > >> > table > >> > with 11 columns and a couple of indexes and triggers) with lack of > >> > feedback. > >> > > >> > -- > >> > Magnus Hagander > >> > Me: http://www.hagander.net/ > >> > Work: http://www.redpill-linpro.com/ > >> > >> > >> > >> -- > >> Dave Page > >> Blog: http://pgsnake.blogspot.com > >> Twitter: @pgsnake > >> > >> EnterpriseDB UK: http://www.enterprisedb.com > >> The Enterprise PostgreSQL Company > > > > > > > > -- > Dave Page > Blog: http://pgsnake.blogspot.com > Twitter: @pgsnake > > EnterpriseDB UK: http://www.enterprisedb.com > The Enterprise PostgreSQL Company > --089e01177095b744fb053d26f153 Content-Type: text/html; charset=UTF-8 Content-Transfer-Encoding: quoted-printable

On Wed, Se= p 21, 2016 at 8:40 PM, Dave Page <dpage@pgadmin.org> wrote:<= br>
Hi

On Wed, Sep 21, 2016 at 2:30 PM, Surinder Kumar
<surinder.kumar@enter= prisedb.com> wrote:
> Hi Dave/Ashesh
>
> The thing we can do is:
> 1) We can display loading message same as other panels if returned res= ponse
> is taking longer than 2 seconds, otherwise we won't display any me= ssage(to
> avoid fluctuations).
> Screenshot - demo_message.png

There's a typo there (laoded)..

> 2) Display a loading spinner along with panel title while data is bein= g
> loaded.
> Screenshot - spinner.png

I don't see a spinner there at all.=C2=A0
=E2=80=8Bsorry its not spinner= , its reload spinning icon in place of properties icon.=E2=80=8B

What about the other places I mentioned?=C2=A0
=C2=A0
- On dialogue open and OK
=C2=A0 =C2=A0 D=E2=80=8B
ispla= y=C2=A0loading message=C2=A0
=E2=80=8Bin dialog till the data loads and on OK we can set m= ouse pointer to 'progress' till the data is not processed.

=E2=80=8B =C2=A0 =C2=A0=E2=80=8B
- On Wizard steps
=E2=80= =8B
=E2=80=8B =C2=A0 = =C2=A0D=E2=80=8B
isplay=C2=A0loading message=C2=A0
=E2=80=8Balso in wizard pages till= it doesn't loads data.
=C2=A0

=E2=80=8B =C2= =A0 =C2=A0=E2=80=8B
- Possibly in the debugger, when stepping?
=E2=80=8B
=E2=80=8B =C2=A0 =C2=A0=E2=80=8B
In case of debugger's step= into, step out & resume, we can set mouse pointer to 'progress'= ;=E2=80=8B
=E2=80=8B. =E2=80=8B

> On Wed, Sep 21, 2016 at 5:53 PM, Dave Page <dpage@pgadmin.org> wrote:
>>
>> Ashesh, Murtuza, Surinder,
>>
>> Some feedback or preferably a patch please? Release is next week!<= br> >>
>> On Fri, Sep 16, 2016 at 3:36 PM, Magnus Hagander <magnus@hagander.net>
>> wrote:
>> > On Fri, Sep 16, 2016 at 3:35 AM, Dave Page <dpage@pgadmin.org> wrote:
>> >>
>> >> Ashesh, Murtuza, Surinder,
>> >>
>> >> Any thoughts on how we can achieve this easily, and where= we're
>> >> lacking notifications? We did have activity notification = in the
>> >> treeview, but that never worked overly well and now I com= e to think of
>> >> it, I don't recall seeing it recently. I'm thinki= ng we need something:
>> >>
>> >> - On treeview node click
>> >>
>> >> - On dialogue open and OK
>> >>
>> >> - On Wizard steps
>> >>
>> >> - Possibly in the debugger, when stepping?
>> >>
>> >> I would think all that we need is to set the mouse cursor= to
>> >> 'progress'.
>> >>
>> >
>> > Actually, the treeview spinner worked most of them time.=C2= =A0 I'm not sure
>> > if it
>> > worked all the time - I didn't check that carefully - but= it did work
>> > most
>> > of the mtime.
>> >
>> > But basically whatever *else* I clicked was the problem. In p= articular
>> > clicking the tabs out right, with the "Properties" = and "SQL" tabs being
>> > the
>> > ones I used and they were extremely slow over the high latenc= y
>> > connection (I
>> > measured almost 4 minutes as the worst time to load a SQL pan= e for a
>> > table
>> > with 11 columns and a couple of indexes and triggers) with la= ck of
>> > feedback.
>> >
>> > --
>> >=C2=A0 Magnus Hagander
>> >=C2=A0 Me: http://www.hagander.net/
>> >=C2=A0 Work: http://www.redpill-linpro.com/
>>
>>
>>
>> --
>> Dave Page
>> Blog: http://pgsnake.blogspot.com
>> Twitter: @pgsnake
>>
>> EnterpriseDB UK: http://www.enterprisedb.com
>> The Enterprise PostgreSQL Company
>
>



--
Dave Page
Blog: http://pgsnake.blogspot.com
Twitter: @pgsnake

EnterpriseDB UK: http://www.enterprisedb.com
The Enterprise PostgreSQL Company

--089e01177095b744fb053d26f153--