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

Hide the terms filter box if there are fewer than 8 terms #10438

Merged
merged 1 commit into from Oct 9, 2018

Conversation

Projects
None yet
3 participants
@notnownikki
Member

notnownikki commented Oct 9, 2018

How has this been tested?

On a site that has fewer than 8 categories, the "Search categories" input should not show. If there are 8 or more, it should.

@notnownikki notnownikki referenced this pull request Oct 9, 2018

Merged

Hierarchical terms sorted by name and filterable #10138

0 of 4 tasks complete

@mtias mtias added this to the 4.0 milestone Oct 9, 2018

@mtias

mtias approved these changes Oct 9, 2018

@notnownikki notnownikki merged commit b869b99 into master Oct 9, 2018

2 checks passed

codecov/project No report found to compare against
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@ZebulanStanphill

This comment has been minimized.

Show comment
Hide comment
@ZebulanStanphill

ZebulanStanphill Oct 9, 2018

Contributor

Does this make sense from an accessibility perspective?

Contributor

ZebulanStanphill commented Oct 9, 2018

Does this make sense from an accessibility perspective?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment