From: | Pramod Ahire <pramod(dot)ahire(at)enterprisedb(dot)com> |
---|---|
To: | Akshay Joshi <akshay(dot)joshi(at)enterprisedb(dot)com> |
Cc: | pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org> |
Subject: | Re: Quick search for menu items & help articles |
Date: | 2021-01-15 07:24:02 |
Message-ID: | 34544069-28B3-4EDE-A2A4-396C048839E2@enterprisedb.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgadmin-hackers |
Hi Akshay,
Thanks for your quick review ! Made changes as per your comments & attached patch with this email.
Pramod Ahire
Software Engineer
C: +91-020-66449600/601
D: +91-9028697679
From: Akshay Joshi <akshay(dot)joshi(at)enterprisedb(dot)com>
Date: Friday, 15 January 2021 at 12:14 PM
To: Pramod Ahire <pramod(dot)ahire(at)enterprisedb(dot)com>
Cc: pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org>
Subject: Re: Quick search for menu items & help articles
Hi Pramod
Following are the review comments:
linter issue "Undefined variable $black"
In Desktop mode remove the separator beside the search icon.
On Thu, Jan 14, 2021 at 10:18 PM Pramod Ahire <pramod(dot)ahire(at)enterprisedb(dot)com> wrote:
Hi Team,
Please find the attached designs & patch that contains complete functionality except below to do for quick search.
To Do:
Unit test cases are not that sufficient to cover complete code, but will be working in background to cover up those one
In pgadmin, for disabled menu items we need to add info that will describe why menu has disabled & how it will be enabled. Either another way to enable all of them & show respective reason in popup that menu is disabled for.
Please do let me know if I missed anything or suggestion of yours.
Thanks !
Pramod Ahire
Software Engineer
C: +91-020-66449600/601
D: +91-9028697679
--
Thanks & Regards
Akshay Joshi
pgAdmin Hacker | Principal Software Architect
EDB Postgres
Mobile: +91 976-788-8246
Attachment | Content-Type | Size |
---|---|---|
quick_search_pgadmin_v2.patch | application/octet-stream | 34.3 KB |
![]() |
image/png | 68.4 KB |
![]() |
image/png | 68.4 KB |
From | Date | Subject | |
---|---|---|---|
Next Message | Akshay Joshi | 2021-01-15 08:33:29 | pgAdmin 4 commit: Added informative message when there is no difference |
Previous Message | Pramod Ahire | 2021-01-15 07:23:24 | Re: Quick search for menu items & help articles |