Skip to content

fix(modal): prevent ion-content collapsing at content-based heights - #31413

Draft
brandyscarney wants to merge 2 commits into
mainfrom
FW-7731
Draft

fix(modal): prevent ion-content collapsing at content-based heights#31413
brandyscarney wants to merge 2 commits into
mainfrom
FW-7731

Conversation

@brandyscarney

@brandyscarney brandyscarney commented Aug 28, 2026

Copy link
Copy Markdown
Member

Issue number: resolves #31149


What is the current behavior?

Setting --height to any of the following on an ion-modal containing an ion-content results in the ion-content collapsing to 0 height: auto, fit-content, min-content, or max-content.

What is the new behavior?

  • Checks if modal --height is set to auto, fit-content, min-content, or max-content and styles the ion-content, ion-nav and .ion-page appropriately if so
  • Updates modal --max-height to 100%
  • Adds a preview test for different scenarios where ion-modal has --height set

Does this introduce a breaking change?

  • Yes
  • No

Other information

Dev build: 9.0.1-dev.11787949960.1e63841c

Preview

@vercel

vercel Bot commented Aug 28, 2026

Copy link
Copy Markdown

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

Project Deployment 操作 Updated (UTC)
ionic-framework Ready Ready Preview Aug 28, 2026 8:49pm

Request Review

注册 for free to join this conversation on GitHub. Already have an account? 登录 to comment

标签

package: core @ionic/core package

项目

None yet

Development

Successfully merging this pull request may close these issues.

bug: ion-modal doesn't show content with auto height

1 participant