Re: [pgAdmin4][Patch]: RM 1780: Rendering glitches in the query editor line number column when editing many lines

From: Khushboo Vashi <khushboo(dot)vashi(at)enterprisedb(dot)com>
To: Dave Page <dpage(at)pgadmin(dot)org>
Cc: pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org>
Subject: Re: [pgAdmin4][Patch]: RM 1780: Rendering glitches in the query editor line number column when editing many lines
Date: 2016-12-16 09:12:24
Message-ID: CAFOhELeR5XJRPCEn7GaWrL5Q-tb_C6eHiqAetXO_dyhonNMSgQ@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgadmin-hackers

Hi,

Please find the attached updated patch.

Thanks,
Khushboo

On Mon, Dec 12, 2016 at 3:30 PM, Dave Page <dpage(at)pgadmin(dot)org> wrote:

> Hi
>
> On Fri, Dec 9, 2016 at 11:37 AM, Khushboo Vashi
> <khushboo(dot)vashi(at)enterprisedb(dot)com> wrote:
> > Hi,
> >
> > Please find the attached patch to fix the RM 1780: Rendering glitches in
> the
> > query editor line number column when editing many lines.
> >
> > Some codemirror css were overwritten to increase the width of the line
> > number column in sql editor.
> > Due to those changes this issue was coming, I have removed those css
> changes
> > and set the line number column to the default size.
> >
> > Please review the patch and let me know if any other UI changes are
> > required.
>
> This seems to work nicely in the query tool, as well as the properties
> pane and for the source code on the properties dialogues for
> functions/views etc.
>
> However, it doesn't seem to work well on the SQL tab of dialogues (see
> the attached screenshot). If you can fix that without changing the
> now-improved behaviour elsewhere, I think this will be a good
> improvement.
>

Fixed

> Thanks.
>
> --
> Dave Page
> Blog: http://pgsnake.blogspot.com
> Twitter: @pgsnake
>
> EnterpriseDB UK: http://www.enterprisedb.com
> The Enterprise PostgreSQL Company
>

Attachment Content-Type Size
RM_1780_ver1.patch text/x-patch 3.5 KB

In response to

Responses

Browse pgadmin-hackers by date

  From Date Subject
Next Message Murtuza Zabuawala 2016-12-16 09:17:52 PATCH: Enhancement in SQL Field Backform control (pgAdmin4)
Previous Message Murtuza Zabuawala 2016-12-16 09:03:09 Re: PATCH: To fix the issue of column order in Primary Key/Index constraints (pgAdmin4)