| 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: | Whole Thread | Raw Message | 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 |
| 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) |