Re: [pgAdmin][RM5154] [Accessibility] : Alertify - Mailing list pgadmin-hackers

From Akshay Joshi
Subject Re: [pgAdmin][RM5154] [Accessibility] : Alertify
Date
Msg-id CANxoLDfG8-OzFDAqumdTrk4KdONTCicw_5Ma-x1nqhRJf+_M0Q@mail.gmail.com
Whole thread Raw
In response to Re: [pgAdmin][RM5154] [Accessibility] : Alertify  (Pradip Parkale <pradip.parkale@enterprisedb.com>)
Responses Re: [pgAdmin][RM5154] [Accessibility] : Alertify  (Pradip Parkale <pradip.parkale@enterprisedb.com>)
List pgadmin-hackers
Thanks, patch applied.

On Tue, Feb 25, 2020 at 3:32 PM Pradip Parkale <pradip.parkale@enterprisedb.com> wrote:
Hi Akshay/Aditya,

Please find the updated patch. I have accommodated all the changes.


On Mon, Feb 24, 2020 at 2:34 PM Aditya Toshniwal <aditya.toshniwal@enterprisedb.com> wrote:
Hi Pradip,

-    commonUtils.findAndSetFocus($(alertifyDialog.elements.body).find('.tab-content div.active'));
+    $(alertifyDialog.elements.dialog).find('[tabindex]:not([tabindex="-1"]').first().focus();


Use the same common function with commonUtils.findAndSetFocus(alertifyDialog.elements.dialog)). It will handle the focus.
Fixed. 
Also, make the required changes mentioned on the pull request - https://github.com/EnterpriseDB/AlertifyJS/pull/1
Fixed. 


On Fri, Feb 21, 2020 at 11:19 AM Akshay Joshi <akshay.joshi@enterprisedb.com> wrote:
Hi Aditya

Can you please review this patch.

On Wed, Feb 19, 2020 at 5:04 PM Pradip Parkale <pradip.parkale@enterprisedb.com> wrote:
Please find the attached patch.

On Wed, Feb 19, 2020 at 5:02 PM Ashesh Vashi <ashesh.vashi@enterprisedb.com> wrote:
Forgot to attach the patch?

On Wed, 19 Feb 2020 at 16:58, Pradip Parkale <pradip.parkale@enterprisedb.com> wrote:
Hi Hackers,

Attached is a patch for accessibility issue related Alertifyjs.
  1. Added hidden label to satisfy accessibility.
  2. Added label with sr-only class.
  3. Added tabindex to move the mouse control to the maximize button using the tab key.

Please note that this patch also includes the fix for the RM-5143 Accessibility issue to maximize the panel for backup and restore windows and all other places.
--
Thanks & Regards,
Pradip Parkale
QMG, EnterpriseDB Corporation
--

--

Thanks & Regards,

Ashesh Vashi
EnterpriseDB INDIA: Enterprise PostgreSQL Company


http://www.linkedin.com/in/asheshvashi



--
Thanks & Regards,
Pradip Parkale
QMG, EnterpriseDB Corporation


--
Thanks & Regards
Akshay Joshi
Sr. Software Architect
EnterpriseDB Software India Private Limited
Mobile: +91 976-788-8246


--
Thanks and Regards,
Aditya Toshniwal
pgAdmin Hacker | Sr. Software Engineer | EnterpriseDB India | Pune
"Don't Complain about Heat, Plant a TREE"


--
Thanks & Regards,
Pradip Parkale
QMG, EnterpriseDB Corporation


--
Thanks & Regards
Akshay Joshi
Sr. Software Architect
EnterpriseDB Software India Private Limited
Mobile: +91 976-788-8246

pgadmin-hackers by date:

Previous
From: Akshay Joshi
Date:
Subject: pgAdmin 4 commit: 1) Added accessibility support in AlertifyJS. Fixes #
Next
From: Akshay Joshi
Date:
Subject: pgAdmin 4 commit: Fixed linter error.