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)

clearfilter.diff (3.4 KB ) - added by fgagnon 13 years ago.
added a 'Remove all' button on top of the FilterBuilder, clearing all filters
295_fixes.diff (5.4 KB ) - added by fgagnon 13 years ago.
This patch apply all suggestions

Download all attachments as: .zip

Change History (6)

by fgagnon, 13 years ago

Attachment: clearfilter.diff added

added a 'Remove all' button on top of the FilterBuilder, clearing all filters

comment:1 by adube, 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.

by fgagnon, 13 years ago

Attachment: 295_fixes.diff added

This patch apply all suggestions

comment:2 by fgagnon, 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 adube, 13 years ago

Milestone: 1.0.0

comment:4 by adube, 13 years ago

Resolution: fixed
Status: newclosed

Fixed in r1144.

Note: See TracTickets for help on using tickets.