Skip to content

chore(deps): update @stencil/core to v4.45.3 - #31738

Merged
brandyscarney merged 1 commit into
mainfrom
FW-7890-stencil
Nov 12, 2026
Merged

brandyscarney merged 1 commit into
mainfrom
FW-7890-stencil

Conversation

@brandyscarney

@brandyscarney brandyscarney commented Nov 11, 2026 •

Copy link
Copy Markdown
Member

Issue number: resolves #31433


What is the current behavior?

Ionic uses @stencil/core 4.45.2. On scoped components, setting innerHTML or innerText removes a marker node Stencil uses to track slotted content, so the content is hidden on the next render.

What is the new behavior?

Does this introduce a breaking change?

  • Yes
  • No

Other information

Regression tests for ion-label are added in #31739, which is stacked on this PR.

@vercel

vercel Bot commented Nov 11, 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 Nov 12, 2026 6:53pm UTC

Request Review

@github-actions github-actions Bot added the package: core @ionic/core package label Nov 11, 2026
@brandyscarney
brandyscarney marked this pull request as ready for review November 11, 2026 22:25
@brandyscarney
brandyscarney requested a review from a team as a code owner November 11, 2026 22:25

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

LGTM! 🚀

@brandyscarney
brandyscarney added this pull request to the merge queue Nov 12, 2026
@github-merge-queue
github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Nov 12, 2026
@brandyscarney
brandyscarney added this pull request to the merge queue Nov 12, 2026
@github-merge-queue
github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Nov 12, 2026
@brandyscarney
brandyscarney added this pull request to the merge queue Nov 12, 2026
@github-merge-queue
github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Nov 12, 2026
@brandyscarney
brandyscarney enabled stacked auto-merge November 12, 2026 18:49
@brandyscarney
brandyscarney added this pull request to the merge queue Nov 12, 2026
Merged via the queue into main with commit 2247098 Nov 12, 2026
56 checks passed
@brandyscarney
brandyscarney deleted the FW-7890-stencil branch November 12, 2026 19:16
ShaneK pushed a commit to masimplo/ionic-framework that referenced this pull request Nov 12, 2026
Issue number: internal

---------

## What is the current behavior?
Binding `innerHTML` (for example Angular's `[innerHTML]`) or setting
`innerText` on an `ion-label` hides the label after its next render.

## What is the new behavior?
- Content set with `innerHTML` or `innerText` renders and stays visible
across re-renders.
- Adds regression tests to `label/test/basic`.

## Does this introduce a breaking change?

- [ ] Yes
- [x] No

## Other information
Stacked on ionic-team#31738,
which updates `@stencil/core` to 4.45.3 and resolves ionic-team#31433. Without
that update these tests fail.

Co-authored-by: Brandy Smith <[email protected]>

This branch was successfully deployed

1 active deployment
Preview — 99b9cdb6 Deployed Nov 12, 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.

bug: ion-label does not render when binding innerHTML directly

2 participants