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

From: Chethana Kumar <chethana(dot)kumar(at)enterprisedb(dot)com>
To: Dave Page <dpage(at)pgadmin(dot)org>
Cc: pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org>, Murtuza Zabuawala <murtuza(dot)zabuawala(at)enterprisedb(dot)com>
Subject: Re: [pgAdmin4][Patch]: Adding connection status in Query tool
Date: 2017-12-22 06:23:55
Message-ID: CAH4-4Wv28z_=R33WrKwh0t12_EG8GRyfFRW0JSj+maA2awN_PQ@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgadmin-hackers

Yes Dave.

Now it is clearly differentiated, please check and confirm.

Thanks,
Chethana kumar

On Fri, Dec 22, 2017 at 12:08 AM, Dave Page <dpage(at)pgadmin(dot)org> wrote:

> Hi
>
> There are 2 icons that look the same except for colour. That’s not going
> to work for some users with colour blindness.
>
> --
> Dave Page
> Blog: http://pgsnake.blogspot.com
> Twitter: @pgsnake
>
> EnterpriseDB UK:http://www.enterprisedb.com
> The Enterprise PostgreSQL Company
>
> On 21 Dec 2017, at 15:32, Chethana Kumar <chethana(dot)kumar(at)enterprisedb(dot)com>
> wrote:
>
> Hi Team,
>
> I have refined the icon once again for database connection status in query
> tool itself.
>
> Below are the 5 different status that I have attached as screenshot -
>
> 1_busy
> 2_error_in_transaction
> 3_valid_transaction_in_idle
> 4_database_connected
> 5_database_disconnected
>
> Please provide your inputs.
>
> Regards,
> Chethana kumar
>
>
> On Tue, Dec 19, 2017 at 5:12 PM, 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
>>
>>
>
>
> --
> Chethana Kumar
> Principal UI/UX Designer
> EnterpriseDB Corporation
>
>
> The Postgres Database Company
>
> P: +91 86981 57146
> www.enterprisedb.com
>
> <1_busy.jpg>
>
> <2_error_in_transaction.jpg>
>
> <3_valid_transaction_in_idle.jpg>
>
> <4_database_connected.jpg>
>
> <5_database_disconnected.jpg>
>
>

--
Chethana Kumar
Principal UI/UX Designer
EnterpriseDB Corporation

The Postgres Database Company

P: +91 86981 57146
www.enterprisedb.com

Attachment Content-Type Size
1_busy.jpg image/jpeg 611.2 KB
2_error_in_transaction.jpg image/jpeg 611.0 KB
3_valid_transaction_in_idle.jpg image/jpeg 611.3 KB
4_database_connected.jpg image/jpeg 611.1 KB
5_database_disconnected.jpg image/jpeg 611.5 KB

In response to

Responses

Browse pgadmin-hackers by date

  From Date Subject
Next Message Murtuza Zabuawala 2017-12-22 06:32:13 [pgAdmin4][Patch]: Do not decrypt the password when the password is 'None'
Previous Message Dave Page 2017-12-21 18:38:01 Re: [pgAdmin4][Patch]: Adding connection status in Query tool