Skip to content

fix(button): prevent container keys ui flash#1033

Merged
shortcuts merged 3 commits intonextfrom
fix/key-container-ui-flash
Jul 23, 2021
Merged

fix(button): prevent container keys ui flash#1033
shortcuts merged 3 commits intonextfrom
fix/key-container-ui-flash

Conversation

@shortcuts
Copy link
Copy Markdown
Member

@shortcuts shortcuts commented Jul 23, 2021

Follow up of #1021

Summary

This PR prevent the UI flash for server-side rendered websites. Since the key value is null, the keys container isn't rendered and therefore causes a UI flash.

Result

The condition is now a level below (on the keys), and we've added a fixed width to the container.

Preview

Screen.Recording.2021-07-23.at.15.47.59.mov

Related

@codesandbox-ci
Copy link
Copy Markdown

codesandbox-ci bot commented Jul 23, 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 bfa436a:

Sandbox Source
Vanilla Configuration

@shortcuts shortcuts marked this pull request as ready for review July 23, 2021 14:02
@shortcuts shortcuts force-pushed the fix/key-container-ui-flash branch from 5389a64 to 751d400 Compare July 23, 2021 14:37
Comment thread packages/docsearch-css/src/button.css Outdated
Comment thread packages/docsearch-css/src/button.css Outdated
Co-authored-by: François Chalifour <francoischalifour@users.noreply.github.com>
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