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

2020-03-25 Thread Nagesh Dhope
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

Re: Fix in switch cell tab navigation

2020-03-25 Thread Ganesh Jaybhay
Hi Khushboo, I have removed the condition added for subgrid-cell. Please find the attached updated patch Regards, Ganesh Jaybhay On Wed, Mar 25, 2020 at 5:39 PM Khushboo Vashi < khushboo.va...@enterprisedb.com> wrote: > Hi Ganesh, > > You have put a check on *subgrid-cell* class, which does

Re: Fix in switch cell tab navigation

2020-03-25 Thread Akshay Joshi
Thanks, patch applied. On Thu, Mar 26, 2020 at 10:30 AM Ganesh Jaybhay < ganesh.jayb...@enterprisedb.com> wrote: > Hi Khushboo, > > I have removed the condition added for subgrid-cell. Please find the > attached updated patch > > Regards, > Ganesh Jaybhay > > On Wed, Mar 25, 2020 at 5:39 PM

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

2020-03-25 Thread Aditya Toshniwal
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,

pgAdmin 4 commit: Fixed an issue where focus doesn't go to the immediat

2020-03-25 Thread Akshay Joshi
Fixed an issue where focus doesn't go to the immediate next editable cell after pressing the tab key on switch cell. Branch -- master Details --- https://git.postgresql.org/gitweb?p=pgadmin4.git;a=commitdiff;h=4c0b229b144a46b3c3b0abf99ebf68f3247d24b4 Author: Ganesh Jaybhay Modified

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

2020-03-25 Thread Nagesh Dhope
Please find an updated patch, which fixes review comments. On Thu, Mar 26, 2020 at 10:24 AM Aditya Toshniwal < aditya.toshni...@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

Fix in switch cell tab navigation

2020-03-25 Thread Ganesh Jaybhay
Hi Hackers, Attached is the patch for below minor fix in the switch cell tab navigation: After pressing the tab key on switch cell, sometimes focus doesn't go to the immediate next editable cell. It goes to the different cell for fraction of seconds and again comes to the next editable cell.

Re: Fix in switch cell tab navigation

2020-03-25 Thread Khushboo Vashi
Hi Ganesh, You have put a check on *subgrid-cell* class, which does not exist in the entire source code. Also, can you please let me know of any module which you have tested, so I can test accordingly though the code for the same looks good to me. Thanks, Khushboo On Wed, Mar 25, 2020 at 3:21