Skip to content

feat(split-pane): resolve breakpoints from config - #31521

Open
brandyscarney wants to merge 2 commits into
FW-7285-4-galleryfrom
FW-7285-5-split-pane
Open

brandyscarney wants to merge 2 commits into
FW-7285-4-galleryfrom
FW-7285-5-split-pane

Conversation

@brandyscarney

@brandyscarney brandyscarney commented Oct 1, 2026 •

Copy link
Copy Markdown
Member

Issue number: internal


What is the current behavior?

The when property accepts a CSS media query, a boolean, or one of the shortcut expressions "xs" through "xl" and "never". The shortcuts map to fixed widths that cannot be changed. The default value is '(min-width: 992px)'.

What is the new behavior?

  • The shortcuts resolve through the global screen breakpoints, so changing a breakpoint with the screenBreakpoints config also changes the width at which the split pane appears.
  • "xxl" is now a supported shortcut.
  • The default value is now the "lg" shortcut. It resolves to the same 992px unless lg is configured otherwise.
  • Media queries, booleans and "never" behave exactly as before.

Does this introduce a breaking change?

  • Yes
  • No

The default value of when changed from '(min-width: 992px)' to 'lg'. The default behavior is unchanged, but code that reads or compares when against the old literal needs updating.

Other information

Preview

@brandyscarney
brandyscarney added this pull request to stack #31503 October 1, 2026 22:43
@vercel

vercel Bot commented Oct 1, 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 6, 2026 7:55pm UTC

Request Review

@github-actions github-actions Bot added the package: core @ionic/core package label Oct 1, 2026
@brandyscarney
brandyscarney marked this pull request as ready for review October 2, 2026 13:50
@brandyscarney
brandyscarney requested a review from a team as a code owner October 2, 2026 13:50
@brandyscarney
brandyscarney requested review from ShaneK and removed request for a team October 2, 2026 13:50

@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 in the tests.

Comment thread core/src/components/split-pane/test/when/split-pane.e2e.ts
Comment thread core/src/components/split-pane/test/when/split-pane.e2e.ts Outdated

This branch was successfully deployed

1 active deployment
Preview — 73196716 Deployed Oct 6, 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.

2 participants