Обсуждение: Bug #2309 fix
Hi,
I am Neethu Mariya Joy, an undergraduate pursuing BE in Computer Science at BITS Pilani.
I've attempted to fix https://redmine.postgresql.org/issues/2309. Codemirrors catches all the keyboard and mouse events when 'readOnly' option is set to 'noCursor' and does not allow copying.
So, I've set 'readOnly' option to true. In order to hide the cursor, I've added a class 'hide-cursor-workaround' and applied css styles to hide the cursor.
I'm attaching my patch as bug2309.diff below as per the contribution guidelines.
Hope this helps. Thank you for your consideration!
Sincerely,
Neethu Mariya Joy
Вложения
Hello Neethu,
We run the patch though our test pipeline and all tests are green. Everything looks good with this patch
Thanks
Joao
On Thu, Mar 1, 2018 at 10:37 AM Neethu Mariya Joy <neethumariyajoy@gmail.com> wrote:
Hi,I am Neethu Mariya Joy, an undergraduate pursuing BE in Computer Science at BITS Pilani.I've attempted to fix https://redmine.postgresql.org/issues/2309. Codemirrors catches all the keyboard and mouse events when 'readOnly' option is set to 'noCursor' and does not allow copying.So, I've set 'readOnly' option to true. In order to hide the cursor, I've added a class 'hide-cursor-workaround' and applied css styles to hide the cursor.I'm attaching my patch as bug2309.diff below as per the contribution guidelines.Hope this helps. Thank you for your consideration!Sincerely,Neethu Mariya Joy
Hi Neethu,
Patch looks good.
We are facing similar issue on dashboard as well where users are not able to copy the sql from server activity table, I've attached screenshot for your reference.
--
Regards,
On Fri, Mar 2, 2018 at 3:35 AM, Joao De Almeida Pereira <jdealmeidapereira@pivotal.io> wrote:
Hello Neethu,We run the patch though our test pipeline and all tests are green. Everything looks good with this patchThanksJoaoOn Thu, Mar 1, 2018 at 10:37 AM Neethu Mariya Joy <neethumariyajoy@gmail.com> wrote:Hi,I am Neethu Mariya Joy, an undergraduate pursuing BE in Computer Science at BITS Pilani.I've attempted to fix https://redmine.postgresql.org/issues/2309. Codemirrors catches all the keyboard and mouse events when 'readOnly' option is set to 'noCursor' and does not allow copying. So, I've set 'readOnly' option to true. In order to hide the cursor, I've added a class 'hide-cursor-workaround' and applied css styles to hide the cursor.I'm attaching my patch as bug2309.diff below as per the contribution guidelines.Hope this helps. Thank you for your consideration!Sincerely,Neethu Mariya Joy
Вложения
Hi,
A similar workaround fixed #3137 too. I'm attaching the patch below.
Sincerely,
Neethu
On Fri, Mar 2, 2018 at 9:45 AM, Murtuza Zabuawala <murtuza.zabuawala@enterprisedb.com> wrote:
Hi Neethu,Patch looks good.We are facing similar issue on dashboard as well where users are not able to copy the sql from server activity table, I've attached screenshot for your reference.--Regards,On Fri, Mar 2, 2018 at 3:35 AM, Joao De Almeida Pereira <jdealmeidapereira@pivotal.io> wrote:Hello Neethu,We run the patch though our test pipeline and all tests are green. Everything looks good with this patchThanksJoaoOn Thu, Mar 1, 2018 at 10:37 AM Neethu Mariya Joy <neethumariyajoy@gmail.com> wrote:Hi,I am Neethu Mariya Joy, an undergraduate pursuing BE in Computer Science at BITS Pilani.I've attempted to fix https://redmine.postgresql.org/issues/2309. Codemirrors catches all the keyboard and mouse events when 'readOnly' option is set to 'noCursor' and does not allow copying. So, I've set 'readOnly' option to true. In order to hide the cursor, I've added a class 'hide-cursor-workaround' and applied css styles to hide the cursor.I'm attaching my patch as bug2309.diff below as per the contribution guidelines.Hope this helps. Thank you for your consideration!Sincerely,Neethu Mariya Joy
Вложения
Hi
Thank you both. I've committed this with a minor tweak to include a space before the { in the CSS, to match the existing formatting.
Welcome to the project Neethu!
--
On Thu, Mar 1, 2018 at 10:05 PM, Joao De Almeida Pereira <jdealmeidapereira@pivotal.io> wrote:
Hello Neethu,We run the patch though our test pipeline and all tests are green. Everything looks good with this patchThanksJoaoOn Thu, Mar 1, 2018 at 10:37 AM Neethu Mariya Joy <neethumariyajoy@gmail.com> wrote:Hi,I am Neethu Mariya Joy, an undergraduate pursuing BE in Computer Science at BITS Pilani.I've attempted to fix https://redmine.postgresql.org/issues/2309. Codemirrors catches all the keyboard and mouse events when 'readOnly' option is set to 'noCursor' and does not allow copying. So, I've set 'readOnly' option to true. In order to hide the cursor, I've added a class 'hide-cursor-workaround' and applied css styles to hide the cursor.I'm attaching my patch as bug2309.diff below as per the contribution guidelines.Hope this helps. Thank you for your consideration!Sincerely,Neethu Mariya Joy
Dave Page
Blog: http://pgsnake.blogspot.com
Twitter: @pgsnake
EnterpriseDB UK: http://www.enterprisedb.com
The Enterprise PostgreSQL Company
Blog: http://pgsnake.blogspot.com
Twitter: @pgsnake
EnterpriseDB UK: http://www.enterprisedb.com
The Enterprise PostgreSQL Company
Thanks, applied.
On Fri, Mar 2, 2018 at 9:05 AM, Neethu Mariya Joy <neethumariyajoy@gmail.com> wrote:
Hi,A similar workaround fixed #3137 too. I'm attaching the patch below.Sincerely,NeethuOn Fri, Mar 2, 2018 at 9:45 AM, Murtuza Zabuawala <murtuza.zabuawala@enterprisedb.com> wrote: Hi Neethu,Patch looks good.We are facing similar issue on dashboard as well where users are not able to copy the sql from server activity table, I've attached screenshot for your reference.--Regards,On Fri, Mar 2, 2018 at 3:35 AM, Joao De Almeida Pereira <jdealmeidapereira@pivotal.io> wrote:Hello Neethu,We run the patch though our test pipeline and all tests are green. Everything looks good with this patchThanksJoaoOn Thu, Mar 1, 2018 at 10:37 AM Neethu Mariya Joy <neethumariyajoy@gmail.com> wrote:Hi,I am Neethu Mariya Joy, an undergraduate pursuing BE in Computer Science at BITS Pilani.I've attempted to fix https://redmine.postgresql.org/issues/2309. Codemirrors catches all the keyboard and mouse events when 'readOnly' option is set to 'noCursor' and does not allow copying. So, I've set 'readOnly' option to true. In order to hide the cursor, I've added a class 'hide-cursor-workaround' and applied css styles to hide the cursor.I'm attaching my patch as bug2309.diff below as per the contribution guidelines.Hope this helps. Thank you for your consideration!Sincerely,Neethu Mariya Joy
Dave Page
Blog: http://pgsnake.blogspot.com
Twitter: @pgsnake
EnterpriseDB UK: http://www.enterprisedb.com
The Enterprise PostgreSQL Company
Blog: http://pgsnake.blogspot.com
Twitter: @pgsnake
EnterpriseDB UK: http://www.enterprisedb.com
The Enterprise PostgreSQL Company