Skip to content

feat(config): add screenBreakpoints config option - #31502

Merged
brandyscarney merged 8 commits into
FW-7285-1-xxlfrom
FW-7285-2-config
Oct 14, 2026
Merged

brandyscarney merged 8 commits into
FW-7285-1-xxlfrom
FW-7285-2-config

Conversation

@brandyscarney

@brandyscarney brandyscarney commented Sep 29, 2026 •

Copy link
Copy Markdown
Member

Adds a shared breakpoints utility that resolves the screen breakpoints from config, validating each value and falling back to the default per key. The xs screen breakpoint defaults to 0 and cannot be changed.

@brandyscarney
brandyscarney added this pull request to stack #31503 September 29, 2026 20:47
@vercel

vercel Bot commented Sep 29, 2026 •

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
ionic-framework Ready Ready Preview Oct 13, 2026 7:37pm UTC

Request Review

@github-actions github-actions Bot added the package: core @ionic/core package label Sep 29, 2026
Comment thread core/src/utils/media.ts Outdated

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

matchBreakpoint was moved to breakpoints.ts since that file name makes it more clear, but I can re-add this file if you think it makes more sense here.

@brandyscarney
brandyscarney marked this pull request as ready for review October 2, 2026 13:49
@brandyscarney
brandyscarney requested a review from a team as a code owner October 2, 2026 13:49
@brandyscarney
brandyscarney requested review from ShaneK and removed request for a team October 2, 2026 13:49

@ShaneK ShaneK 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 to me, great work! Just a couple of optional nits, no worries if you'd rather leave them.

Comment thread core/src/utils/config.ts
Comment thread core/src/utils/test/breakpoints.spec.ts Outdated
Comment thread core/src/index.ts Outdated
Comment thread core/src/utils/breakpoints.ts Outdated
Comment thread core/src/utils/breakpoints.ts
Comment thread core/src/global/ionic-global.ts Outdated

@thetaPC thetaPC 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.

LGTM

@brandyscarney
brandyscarney merged commit a56dc10 into FW-7285 Oct 14, 2026
54 checks passed
@brandyscarney
brandyscarney deleted the FW-7285-2-config branch October 14, 2026 14:08
brandyscarney added a commit that referenced this pull request Oct 14, 2026
Adds a shared breakpoints utility that resolves the screen breakpoints
from config, validating each value and falling back to the default per
key. The `xs` screen breakpoint defaults to `0` and cannot be changed.

---------

Co-authored-by: Brandy Smith <6577830+brandyscarney@users.noreply.github.com>

This branch was successfully deployed

1 active deployment
Preview — af364198 Deployed Oct 13, 2026 by vercel[bot]
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

package: core @ionic/core package

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants