Re: [pgAdmin4][Patch]: Adding connection status in Query tool

Hi

Interesting. A few thoughts:

- The pulsating icon is very off-putting. I think we need to make it only
flash a couple of times when we actually need to attract the attention of
the user.

- We shouldn't really use tooltips like this, as it may confuse folks with
screen readers. Should we make the icon clickable (which should have a
visual hint)? Maybe a drop-down status panel.

- Do we need to poll separately for the status? Instead, why not update it
whenever polling for results, or executing something?

Thanks!

On Tue, Dec 19, 2017 at 11:42 AM, Murtuza Zabuawala <murtuza.zabuawala@
enterprisedb.com> wrote:

> Hi,
>
> PFA patch to add the connection status
> <http://initd.org/psycopg/docs/extensions.html#transaction-status-constants>
> in query tool, this feature will allow user to check the database
> connection status in query tool itself, it will also provide the detailed
> status as a tooltip when user hovers on it, the most benefit of the feature
> will be when user open query tool in new Browser Tab where Browser tree is
> not visible to user, user can also configure the status polling time using
> preference dialog.
> RM#2475
>
> Apart from that I have also removed the ..sqleditor/static/css/sqledit
> or.css reference from ../datagrid/templates/datagrid/index.html file
> because we are already bundling the "sqleditor.css" file in main
> "style.css" file.
>
>
> Thanks to Chethana for his UI related inputs and to Surinder for helping
> me on html alignment issues.
>
>
> --
> Regards,
> Murtuza Zabuawala
> EnterpriseDB: 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