Skip to content
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

[559] Show scopes if no results are found #1804

Merged
merged 1 commit into from
Jan 15, 2013
Merged

[559] Show scopes if no results are found #1804

merged 1 commit into from
Jan 15, 2013

Conversation

seanlinsley
Copy link
Contributor

This PR reverts #559 and is a half-fix for #995

Since we currently hide/remove scopes when a filter doesn't return results, it's much more difficult for the user to figure out "what went wrong" with their query. If they can see that all available scopes are reporting 0 records for the query, they know with a much higher degree of certainty that there truly is nothing to find. What's worse, since the scopes are currently hidden, there would be no way to know if there were results in other scopes!

This PR is tangentially related to #1532. I intend to implement "minimap" of sorts to explicitly tell the user how their result set has been refined:

When using filters, the previous behavior was to hide all scopes. This
is counterintuitive, as a different scope from the current one may have
results for the selected filter, and ends up confusing the user.
@pcreux
Copy link
Contributor

pcreux commented Dec 7, 2012

I prefere this behavior too. Anyone against that?

gregbell added a commit that referenced this pull request Jan 15, 2013
[559] Show scopes if no results are found
@gregbell gregbell merged commit 8759692 into activeadmin:master Jan 15, 2013
@seanlinsley seanlinsley deleted the 559-show-scopes-if-no-result branch February 8, 2013 01:08
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