Skip to content

The Clear All Filters button is converted to a checkable button. #1235

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
wants to merge 1 commit into from
Closed

The Clear All Filters button is converted to a checkable button. #1235

wants to merge 1 commit into from

Conversation

justinclift
Copy link
Member

When filters are applied, this button clears them as before,
but it also saves the current filter values for later restoring.

When no filters are applied, it restores the last used filters in
the table, if previously saved.

The button also provides a visual hint (check state) of whether
the filters are applied in any column (they may be out of the
view).

The icon has been updated in line with the new usage.

When filters are applied, this button clears them as before,
but it also saves the current filter values for later restoring.

When no filters are applied, it restores the last used filters in
the table, if previously saved.

The button also provides a visual hint (check state) of whether
the filters are applied in any column (they may be out of the
view).

The icon has been updated in line with the new usage.
@justinclift
Copy link
Member Author

justinclift commented Nov 20, 2017

@mgrojo This is your PR from #1214, with some minor cleanups:

  • The commit history was a bit of a mess (nothing bad, just normal early learning stuff 😉) so I've manually rebased this onto our latest git commit
  • The icon.qrc patch file wasn't merging automatically due to changes in the file since the PR. Updated to merge ok now.
  • There were a few window size and default tab changes in the original version, which I've deleted without looking at closely. (blindly assuming they aren't needed, but might be wrong 😉)

In theory this should be ok for review now. 😄

@MKleusberg
Copy link
Member

Sorry for the delay, I will review this next weekend 😃

@mgrojo
Copy link
Member

mgrojo commented Nov 22, 2017

Thank you, @justinclift. The window size and default tab changes were totally unintended. The usual staff performed by qtcreator, that initially I was afraid to undo, thinking it might be needed by the manual changes I've done. I know now that I should revert all those lines.

By the way, one or two of my latest commits were performed using the "git pull --rebase". It works like a charm. 😄

@justinclift
Copy link
Member Author

Awesome. 😄

@justinclift
Copy link
Member Author

@mgrojo This PR was superseded by a later one wasn't it? Just thinking we should close this now if it was. 😄

@mgrojo
Copy link
Member

mgrojo commented Dec 4, 2017

Yes, with the other PR we can save filters in a more powerful way. We can close this as superseded by #1246

@mgrojo mgrojo closed this Dec 4, 2017
@MKleusberg
Copy link
Member

Cool 😄

I was just starting to write my thoughts about this. So for the record: I can see how it is useful for some use cases but rather irritating for others. Let's say I'm looking up some data in a large table to copy it into some other application. My workflow could be to set some filters to narrow down the table, then copy my field, then hit the clear filters button, and start over with step 1. And then repeat this again and again. That's possible with the current approach (and one reason we added the clear filters button) but doesn't work with the toggle-able (is that a word?) clear filter button. So we would have had to add two buttons or something to have both approaches still working...

@justinclift justinclift deleted the toggle_filters2 branch December 6, 2017 16:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants