Opened 13 years ago
Closed 13 years ago
#295 closed enhancement (fixed)
WFSFilterBuilder - Clear filter
Reported by: | fgagnon | Owned by: | fgagnon |
---|---|---|---|
Priority: | major | Milestone: | 1.0.0 |
Component: | widgets | Version: | |
Keywords: | wfsfilterbuilder | Cc: |
Description
Since the filter built is kept in memory when you close the window, adding a button to reset the FilterBuilder (and empty the search textfield of the result panel) would be useful.
Attachments (2)
Change History (6)
by , 13 years ago
Attachment: | clearfilter.diff added |
---|
comment:1 by , 13 years ago
Here's a review of this patch :
- The button resets both FilterBuilder components (on the left) and the search textfield (on the right). I think it would be more appropriate to have it accessible from both tabs, thus positioning it in the top toolbar, next to the layer list.
- if we do that, its label could be rename from "Remove all" to "Reset all filters"
- I think it should trigger a new request upon click as well. Currently, clicking resets everything but the result grid, which is odd. In doubt, we could have an flag option for that.
comment:2 by , 13 years ago
The latest patch moved the new button to main toolbar beside the layer selector. The label was renamed as well.
In addition, it now triggers a new request correctly upon click. Previously, it only did so when the result panel (with the grid.Search) was active. As a side effect, this also fix an unreported bug where the search textfield was not cleared when the layer was changed, despite its associated filter being reset.
Additional testing is highly recommended.
comment:3 by , 13 years ago
Milestone: | → 1.0.0 |
---|
added a 'Remove all' button on top of the FilterBuilder, clearing all filters