Skip to content

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

Merged
shortcuts merged 2 commits into
algolia:nextfrom
tnir:tnir-css-de-facto-breakpoints
Jul 21, 2022
Merged

feat(styles): set medium breakpoint to standarized value#1446
shortcuts merged 2 commits into
algolia:nextfrom
tnir:tnir-css-de-facto-breakpoints

Conversation

@tnir

@tnir tnir commented Jul 21, 2022

Copy link
Copy Markdown
Contributor

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

netlify Bot commented Jul 21, 2022

Copy link
Copy Markdown

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

codesandbox-ci Bot commented Jul 21, 2022

Copy link
Copy Markdown

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

@shortcuts shortcuts left a comment

Copy link
Copy Markdown
Member

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

@shortcuts shortcuts left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Looks good, thanks!

@tnir

tnir commented Jul 21, 2022

Copy link
Copy Markdown
Contributor Author

@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