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

Поиск
Список
Период
Сортировка
От Dave Page
Тема Re: [pgAdmin4][Patch]: Adding connection status in Query tool
Дата
Msg-id CA+OCxowRVM=CuhKzpe0LGLArkcyK6czBOtMQb1EMEDy3ZiHFMw@mail.gmail.com
обсуждение исходный текст
Ответ на [pgAdmin4][Patch]: Adding connection status in Query tool  (Murtuza Zabuawala <murtuza.zabuawala@enterprisedb.com>)
Ответы Re: [pgAdmin4][Patch]: Adding connection status in Query tool  (Murtuza Zabuawala <murtuza.zabuawala@enterprisedb.com>)
Список pgadmin-hackers
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 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/sqleditor.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

В списке pgadmin-hackers по дате отправления:

Предыдущее
От: Dave Page
Дата:
Сообщение: Re: [pgAdmin4][Patch]: RM #2964 - [Desktop Runtime] pgAdmin4 crash ifuser download query as CVS
Следующее
От: Murtuza Zabuawala
Дата:
Сообщение: Re: [pgAdmin4][Patch]: Adding connection status in Query tool