Re: [pgAdmin][RM4310] User can not connect to the Master Passwordwith Enter button

Поиск
Список
Период
Сортировка
От Aditya Toshniwal
Тема Re: [pgAdmin][RM4310] User can not connect to the Master Passwordwith Enter button
Дата
Msg-id CAM9w-_mn7mBrAcpBqMayPPU-btXenfX5+YKmsp_8w5qFd0903w@mail.gmail.com
обсуждение исходный текст
Ответ на Re: [pgAdmin][RM4310] User can not connect to the Master Passwordwith Enter button  (Dave Page <dpage@pgadmin.org>)
Ответы Re: [pgAdmin][RM4310] User can not connect to the Master Passwordwith Enter button  (Dave Page <dpage@pgadmin.org>)
Список pgadmin-hackers
Hi,

On Wed, Jun 5, 2019 at 4:48 PM Dave Page <dpage@pgadmin.org> wrote:
Hi

On Wed, Jun 5, 2019 at 8:14 AM Aditya Toshniwal <aditya.toshniwal@enterprisedb.com> wrote:
Hi Hackers,

Attached is the patch to fix an issue where using browser autofills in alertify dialogs triggers the help page button (#4317).
A workaround was added to master password dialog to avoid this, but that in turn disabled the 'OK' button tigger on pressing enter button. This is also fixed with this patch.(#4310)

This isn't quite right. When the browser auto-fills the password, the OK button remains disabled. I can hit enter though - however, I shouldn't be able to do that when the OK button is disabled (being the default button on the dialogue, it should respond to enter, unless another control which also accepts enter has focus (e.g. a multiline text area).
The autofilled password appears as set, but it is actually not set to textbox unless you enter/click in the dropdown. We get the DOM element value as blank even if it appears filled. This is how chromium autofill behaves. Upon hitting enter the autofill value is set to the textbox and OK button is enabled.

The OK button should be enabled - probably all the time, as the user could have entered a blank password, and most systems won't prevent a user from trying to use such a password.
I have made the changes to allow blank master password, plus enabling OK button always. Attached is the updated patch.

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

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


--
Thanks and Regards,
Aditya Toshniwal
Software Engineer | EnterpriseDB India | Pune
"Don't Complain about Heat, Plant a TREE"
Вложения

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

Предыдущее
От: Aditya Toshniwal
Дата:
Сообщение: Re: [pgAdmin][RM4195] Tab navigation is broken in Query tool (Accessibility)
Следующее
От: Aditya Toshniwal
Дата:
Сообщение: [pgAdmin][Doc] Master password doc change to add reference to desktop deployment