Re: [pgAdmin4][Patch]: RM#1242 - Cleanup busy indication for tree view and tab panes

From: Surinder Kumar <surinder(dot)kumar(at)enterprisedb(dot)com>
To: Dave Page <dpage(at)pgadmin(dot)org>
Cc: pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org>
Subject: Re: [pgAdmin4][Patch]: RM#1242 - Cleanup busy indication for tree view and tab panes
Date: 2016-06-27 13:38:36
Message-ID: CAM5-9D-swkGYB43kBn62bY7ETknT0R10im3FgqJ1wZHtELO8+Q@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgadmin-hackers

Hi Dave,

*Following are the changes/fixes in the patch:*

1. The spinner icons for the Query Tool and Debugger are now consistent.
2. Greyed background is now consistent throughout the panels.
3. Text is centered.
4. Now CSS styles are generic for spinner in all panels.
5. Panes content is now refreshed only when new data differs from
previous data, reused the code written in dashboard js
6. Add spinner to the data grid and query tool when they opened from the
context menu.
7. Hide spinner after 1000ms so that it doesn't seems to cause
flickering.
8. Bigger spinner icon is removed from the treeview.

Please review.

On Fri, Jun 24, 2016 at 3:04 PM, Dave Page <dpage(at)pgadmin(dot)org> wrote:

> On Thu, Jun 23, 2016 at 1:52 PM, Surinder Kumar
> <surinder(dot)kumar(at)enterprisedb(dot)com> wrote:
> > Hi
> >
> > Following are the changes in patch:
> >
> > 1) Assign a common class 'pg_spinner' for loading spinner in wcDocker for
> > all panels.
> > 2) Removed the default wcDocker spinner icon which was specific to page
> > load.
> > 3) In case of collection node, we were re-creating the grid every time
> new
> > data comes. instead it should only reset the collection with new data.
> > 4) Move the code to hide the msgContainer and display grid outside the
> ajax
> > which was causing flicker into the dependent panel.
> > 5) Set the timeout for hiding the spinner to 500 inside the success
> callback
> > to stop flicker because response is received immediately and cause
> flicker.
> >
> > Not Fixed:
> > Panes should have their content refreshed until and unless the new data
> > differs from the previous, to reduce flicker.
> > I am working on this issue.
> >
> > Please review the patch.
>
> I think this is an improvement, but not yet ready. For example:
>
> - The treeview shows spinners on the junctions of the branches, and a
> bigger one in the middle of the pane, simultaneously. I think the
> bigger one should be removed.
>
> - The spinners for the Query Tool and Debugger are inconsistent and
> poorly laid out:
> - Different spinner icons (FYI, I prefer the one used in the
> debugger - can we use that everywhere?)
> - Text not centered
> - Background greyed in the Query Tool, but not the Debugger
>
> Additionally, I've logged another issue
> (https://redmine.postgresql.org/issues/1400) which you may want to
> take care of at the same time as this.
>
> Thanks!
>
> --
> Dave Page
> Blog: http://pgsnake.blogspot.com
> Twitter: @pgsnake
>
> EnterpriseDB UK: http://www.enterprisedb.com
> The Enterprise PostgreSQL Company
>

Attachment Content-Type Size
RM#1242_v1.patch application/octet-stream 24.2 KB

In response to

Responses

Browse pgadmin-hackers by date

  From Date Subject
Next Message Dave Page 2016-06-27 16:05:26 Re: PATCH: Fix the issue for saving query output as CSV
Previous Message Murtuza Zabuawala 2016-06-27 12:08:30 PATCH: Fix the issue for saving query output as CSV