search

apache / incubator-ponymail

Apache Pony Mail (Incubating) - Email for Ponies & People
http://ponymail.incubator.apache.org/
Other
80 stars 30 forks source link

Add "clear filters" option #3

Closed ulises closed 9 years ago

ulises commented 9 years ago

I understand that going back a page would undo the last filter.

Ideally this should be more explicit, and it should clear all filters applied, not just the last.

Humbedooh commented 9 years ago

There is only ever one filter applied to a search, which is the current filter. Going back applies the old filter, going forward applies the newer one, but not on the data filtered by the old filter. So I don't quite see the issue here.

ulises commented 9 years ago

Going back to undo the filter is not documented (granted, nothing is atm). I'd personally find it easier to use if there was a way to undo filtering right on the screen.

As mentioned in the ponymail thread: the behaviour is a mixture of webpage apps with webapps (the distinction being that some operations are done via clickity buttons and others via browser operations, e.g. navigating back). Some unification here would be good.

Humbedooh commented 9 years ago

Does 928d593 address your concerns?

ulises commented 9 years ago

does fix!

Humbedooh commented 9 years ago

closing as fixed then!