Skip to content

fix(js): ignore empty template with no query and openOnFocus#407

Merged
francoischalifour merged 5 commits intonextfrom
fix/emptyStates
Jan 27, 2021
Merged

fix(js): ignore empty template with no query and openOnFocus#407
francoischalifour merged 5 commits intonextfrom
fix/emptyStates

Conversation

@shortcuts
Copy link
Copy Markdown
Member

Summary

When openOnFocus is false, the empty source template or renderEmpty was still rendered.

@codesandbox-ci
Copy link
Copy Markdown

codesandbox-ci Bot commented Jan 22, 2021

This pull request is automatically built and testable in CodeSandbox.

To see build info of the built libraries, click here or the icon next to each commit SHA.

Latest deployment of this branch, based on commit 818f7fb:

Sandbox Source
@algolia/js-example Configuration
@algolia/react-renderer-example Configuration

Comment thread packages/autocomplete-js/src/autocomplete.ts Outdated
Comment thread packages/autocomplete-js/src/__tests__/autocomplete.test.ts Outdated
Comment thread packages/autocomplete-js/src/__tests__/autocomplete.test.ts Outdated
Comment thread packages/autocomplete-js/src/__tests__/autocomplete.test.ts Outdated
Comment thread packages/autocomplete-js/src/__tests__/autocomplete.test.ts
Comment thread packages/autocomplete-js/src/__tests__/autocomplete.test.ts Outdated
@francoischalifour francoischalifour changed the title fix(emptyStates): fix openOnFocus priority over rendering empty templates fix(js): fix openOnFocus priority over rendering empty templates Jan 27, 2021
@francoischalifour francoischalifour changed the title fix(js): fix openOnFocus priority over rendering empty templates fix(js): ignore empty template with no query and openOnFocus Jan 27, 2021
@francoischalifour francoischalifour merged commit 92eeb3e into next Jan 27, 2021
@francoischalifour francoischalifour deleted the fix/emptyStates branch January 27, 2021 13:52
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