Re: [pgAdmin4][Patch]: RM #3607 Edit Data: Not able to remove filterSQL condition
On Tue, Sep 11, 2018 at 7:10 PM, Dave Page <dpage@pgadmin.org> wrote:
HiOn Tue, Sep 11, 2018 at 10:12 AM, Akshay Joshi <akshay.joshi@enterprisedb.com> wrote: Hi Hackers,Attached is the patch to fix the following for filter dialog:
- OK button should be disabled for Filter dialog when filter string is empty. Filter dialog should be opened using context menu View Data -> Filtered Rows or shortcut toolbar.
- On validation error dialog should not be closed.
- User should be able to clear the filter string from the Query Tool.
- Validation error should be thrown when filter dialog opened from Query Tool and provide some wrong filter string.
Please review it.This works better now, but I think there are still some related issues. If I clear the filter, then the filter/sort button stays blue in this mode; and that is because we sort by default.However; when we open in first/last/view modes, we also sort by default, but *don't* colour the button.I think that the button should always be blue if there is sorting and/or filtering in place, and that the standard "View" option should *not* sort at all by default.Make sense?
Attached is the modified patch where I have added following:
- When user click on View Data -> All Row, data will not sorted by default, when filter dialog opens from query tool it won't show anything (empty dialog).
- When user click on View Data -> All Row, and later change the limit then data will be sorted based on primary keys.
- When user click on View Data -> First/Last 100 rows then it will be sorted on primary keys and when filter dialog opens from query tool then it shows the sorting columns.
--Dave Page
Blog: http://pgsnake.blogspot.com
Twitter: @pgsnake
EnterpriseDB UK: http://www.enterprisedb.com
The Enterprise PostgreSQL Company
Akshay Joshi
Sr. Software Architect
Phone: +91 20-3058-9517
Mobile: +91 976-788-8246
Mobile: +91 976-788-8246
Вложения
В списке pgadmin-hackers по дате отправления: