Skip to content

refactor(v2): show search icon only on mobiles#2791

Merged
yangshun merged 1 commit into
masterfrom
lex111/search-icon-mobile
May 22, 2020
Merged

refactor(v2): show search icon only on mobiles#2791
yangshun merged 1 commit into
masterfrom
lex111/search-icon-mobile

Conversation

@lex111

@lex111 lex111 commented May 22, 2020

Copy link
Copy Markdown
Contributor

Motivation

Remastered version of #2702, closes #2699

Have you read the Contributing Guidelines on pull requests?

Yes

Test Plan

ezgif com-video-to-gif (3)

Now the search input width is adaptive, it changes depending on the width of the viewport, see the screenshot below:

screenshot_4

Related PRs

(If this PR adds or changes functionality, please take some time to update the docs at https://github.com/facebook/docusaurus, and link to your PR here.)

@lex111 lex111 added the pr: polish This PR adds a very minor behavior improvement that users will enjoy. label May 22, 2020
@lex111 lex111 requested a review from yangshun as a code owner May 22, 2020 15:27
@facebook-github-bot facebook-github-bot added the CLA Signed Signed Facebook CLA label May 22, 2020
@docusaurus-bot

Copy link
Copy Markdown
Contributor

Deploy preview for docusaurus-2 ready!

Built with commit 2499de7

https://deploy-preview-2791--docusaurus-2.netlify.app

@yangshun yangshun left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Awesome thank you!

@yangshun yangshun added the pr: breaking change Existing sites may not build successfully in the new version. Description contains more details. label May 22, 2020
@yangshun yangshun merged commit 83f980c into master May 22, 2020
@yangshun yangshun deleted the lex111/search-icon-mobile branch May 22, 2020 21:02
@lex111 lex111 added this to the v2.0.0-alpha.56 milestone May 27, 2020
mrizwanashiq pushed a commit to mrizwanashiq/docusaurus that referenced this pull request Jun 25, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA Signed Signed Facebook CLA pr: breaking change Existing sites may not build successfully in the new version. Description contains more details. pr: polish This PR adds a very minor behavior improvement that users will enjoy.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

alpha-54: Site title no longer wraps - search box moves

4 participants