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

feat(styles): set medium breakpoint to standarized value #1446

Merged
merged 2 commits into from Jul 21, 2022

Conversation

tnir
Copy link
Contributor

@tnir tnir commented Jul 21, 2022

768px has been much more common for many years (since 2010), and other major CSS frameworks have used 768px as the breakpoint for years.

Closes #1444

Signed-off-by: Takuya Noguchi takninnovationresearch@gmail.com

@netlify
Copy link

netlify bot commented Jul 21, 2022

Deploy Preview for docsearch canceled.

Name Link
🔨 Latest commit 9d2b83b
🔍 Latest deploy log https://app.netlify.com/sites/docsearch/deploys/62d93dc72516900008696760

@tnir tnir force-pushed the tnir-css-de-facto-breakpoints branch from 3fbd52d to 9825604 Compare July 21, 2022 08:08
768px has been much more common for many years (since 2010), and
other major CSS frameworks have used 768px as the breakpoint for
years.

Signed-off-by: Takuya Noguchi <takninnovationresearch@gmail.com>
@tnir tnir force-pushed the tnir-css-de-facto-breakpoints branch from 9825604 to b36b061 Compare July 21, 2022 08:10
@codesandbox-ci
Copy link

codesandbox-ci bot commented Jul 21, 2022

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 9d2b83b:

Sandbox Source
Vanilla Configuration

Copy link
Member

@shortcuts shortcuts left a comment

Choose a reason for hiding this comment

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

Thanks! It make sense indeed since it's standarized.

Could you group all of the changes in this PR please? It would make more sense in the changelog and if we need to point this change later.

Signed-off-by: Takuya Noguchi <takninnovationresearch@gmail.com>
@tnir tnir changed the title feat(css): set breakpoints the de facto value feat(css/react): set breakpoints the de facto value Jul 21, 2022
Copy link
Member

@shortcuts shortcuts left a comment

Choose a reason for hiding this comment

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

Looks good, thanks!

@tnir
Copy link
Contributor Author

tnir commented Jul 21, 2022

@shortcuts Thank you. Both commits are now in this PR. Could you change the commit message with the below (or whatever you like) when you squash-merge?

feat(css/react): set breakpoints the de facto value

@shortcuts shortcuts changed the title feat(css/react): set breakpoints the de facto value feat(styles): set medium breakpoint to standarized value Jul 21, 2022
@shortcuts shortcuts merged commit 8e2e90e into algolia:next Jul 21, 2022
@tnir tnir deleted the tnir-css-de-facto-breakpoints branch July 21, 2022 12:05
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.

[docsearch-css] Breakpoint is to be more de facto value
2 participants