Thread: [pgAdmin][RM5292] Alertify header buttons focus color issue

[pgAdmin][RM5292] Alertify header buttons focus color issue

From
Nagesh Dhope
Date:
Hi Hackers,

Please find a patch which includes
  1. Use btn-secondary styles for all buttons in alertify dialog header.
  2. Added !default for $active-border variable, so that it can be overridden if required. 
  3. Added entry for .scss in .editorconfig
--
Thanks,
Nagesh
Attachment

Re: [pgAdmin][RM5292] Alertify header buttons focus color issue

From
Aditya Toshniwal
Date:
Hi Nagesh,

I cannot see any change related to $active-border variable apart from !default. Plus, $active-border should not be allowed to edit. It is a structural variable and not a variable which affects theme change.
Please revert the $active-border change and send the patch.

On Thu, Mar 26, 2020 at 10:14 AM Nagesh Dhope <nagesh.dhope@enterprisedb.com> wrote:
Hi Hackers,

Please find a patch which includes
  1. Use btn-secondary styles for all buttons in alertify dialog header.
  2. Added !default for $active-border variable, so that it can be overridden if required. 
  3. Added entry for .scss in .editorconfig
--
Thanks,
Nagesh


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

Re: [pgAdmin][RM5292] Alertify header buttons focus color issue

From
Nagesh Dhope
Date:
Please find an updated patch, which fixes review comments.

On Thu, Mar 26, 2020 at 10:24 AM Aditya Toshniwal <aditya.toshniwal@enterprisedb.com> wrote:
Hi Nagesh,

I cannot see any change related to $active-border variable apart from !default. Plus, $active-border should not be allowed to edit. It is a structural variable and not a variable which affects theme change.
Please revert the $active-border change and send the patch.

On Thu, Mar 26, 2020 at 10:14 AM Nagesh Dhope <nagesh.dhope@enterprisedb.com> wrote:
Hi Hackers,

Please find a patch which includes
  1. Use btn-secondary styles for all buttons in alertify dialog header.
  2. Added !default for $active-border variable, so that it can be overridden if required. 
  3. Added entry for .scss in .editorconfig
--
Thanks,
Nagesh


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


--
Thanks,
Nagesh
Attachment

Re: [pgAdmin][RM5292] Alertify header buttons focus color issue

From
Akshay Joshi
Date:
Thanks, patch applied.

On Thu, Mar 26, 2020 at 11:00 AM Nagesh Dhope <nagesh.dhope@enterprisedb.com> wrote:
Please find an updated patch, which fixes review comments.

On Thu, Mar 26, 2020 at 10:24 AM Aditya Toshniwal <aditya.toshniwal@enterprisedb.com> wrote:
Hi Nagesh,

I cannot see any change related to $active-border variable apart from !default. Plus, $active-border should not be allowed to edit. It is a structural variable and not a variable which affects theme change.
Please revert the $active-border change and send the patch.

On Thu, Mar 26, 2020 at 10:14 AM Nagesh Dhope <nagesh.dhope@enterprisedb.com> wrote:
Hi Hackers,

Please find a patch which includes
  1. Use btn-secondary styles for all buttons in alertify dialog header.
  2. Added !default for $active-border variable, so that it can be overridden if required. 
  3. Added entry for .scss in .editorconfig
--
Thanks,
Nagesh


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


--
Thanks,
Nagesh


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