Opened 13 years ago
Last modified 13 years ago
#357 new defect
WFSFilterBuilder - search field broke grid after too many request
| Reported by: | cdeschenes | Owned by: | belug |
|---|---|---|---|
| Priority: | major | Milestone: | 1.8.0 |
| Component: | widgets | Version: | trunk |
| Keywords: | WFSFilterBuilder, seach, delay, request, field | Cc: | adube |
Description
When user do search in WFSFilterBuilder results tab, the list stop respond when user change search field when another search request is already pending. After this, there is nothing to do to repair this.
Possible solutions
- Increase delay after key press to send request (currently hardcoded to 300 in Ext.ux.grd.Search componement)
- set Search field disabled when request is already pending.
Attachments (6)
Change History (17)
by , 13 years ago
| Attachment: | patch-wfsfilterbuilder-searchfield-357-r1177-A0.diff added |
|---|
by , 13 years ago
| Attachment: | patch-wfsfilterbuilder-searchfield-357-r1177-A1.diff added |
|---|
Another fix
by , 13 years ago
| Attachment: | patch-wfsfilterbuilder-searchfield-357-r1177-A2.diff added |
|---|
I forget a debug code in revision A1
by , 13 years ago
| Attachment: | patch-wfsfilterbuilder-searchfield-357-r1205-A3.diff added |
|---|
Update of the fix
by , 13 years ago
| Attachment: | patch-wfsfilterbuilder-searchfield-357-r1206-A4.diff added |
|---|
Same fix, property name changed, minor corrections
comment:4 by , 13 years ago
cdeschenes, here's an updated version of your patch with the following minor fixes :
- property "renderDelay" renamed "triggerDelay"
- English corrections
issue : I noticed that the patch introduces a small bug. When disabling the search text field, the cursor disappear. Clicking in it doesn't bring it back. Please make the according fixes.
by , 13 years ago
| Attachment: | patch-wfsfilterbuilder-searchfield-357-r1217-A5.diff added |
|---|
Fix for cursor (disabled replaced by readonly state)
comment:6 by , 13 years ago
| Resolution: | → fixed |
|---|---|
| Status: | new → closed |
Fixed in r1223, p=cdeschenes r=me.
comment:7 by , 13 years ago
| Resolution: | fixed |
|---|---|
| Status: | closed → reopened |
| Version: | → trunk |
The fix doesn't work in IE.
comment:8 by , 13 years ago
In IE, while the query is in progress and the search text field gets grayed out, you can still enter characters using the keyboard, thus launching more requests.
comment:9 by , 13 years ago
| Milestone: | 1.2.0 → 1.4.0 |
|---|
No patch : pushed to 1.4 since the widget is still considered unstable.
comment:11 by , 13 years ago
| Milestone: | 1.6.0 → 1.8.0 |
|---|---|
| Owner: | changed from to |
| Status: | reopened → new |
The proposed fix is still incomplete. I'm pushing this to 1.8 and reassigning it to belug.
A fix disabling search fields during requests