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

Allow deselecting a facet by clicking the facet value also in AND facets #300

Merged
merged 1 commit into from
Feb 19, 2015

Conversation

EreMaijala
Copy link
Contributor

Our usability studies showed that users expect they can toggle a facet on and off by clicking it. This already works for OR facets, but these changes allow delection of a facet by clicking it in the facet list also for AND facets.

demiankatz added a commit that referenced this pull request Feb 19, 2015
Allow deselecting a facet by clicking the facet value also in AND facets
@demiankatz demiankatz merged commit c90f3bc into vufind-org:master Feb 19, 2015
@demiankatz
Copy link
Member

Makes perfect sense -- thanks!

@demiankatz
Copy link
Member

Just had a thought about this one: do you also use the top facet bar, or are you only using the side facets? It probably makes sense to add the same functionality to the top bar. Do you have time for this, or should I look into it?

@EreMaijala
Copy link
Contributor Author

Good point. We don't use the top facet bar, so I'd appreciate if you could look into it.

@demiankatz
Copy link
Member

Thanks -- Chris or I will take care of it shortly.

@EreMaijala EreMaijala deleted the facet-click branch May 4, 2015 08: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.

2 participants