Skip to content

fix(avatar): fix avatar disabled for icon content#30820

Merged
JoaoFerreira-FrontEnd merged 3 commits into
nextfrom
ROU-12319
Dec 3, 2025
Merged

fix(avatar): fix avatar disabled for icon content#30820
JoaoFerreira-FrontEnd merged 3 commits into
nextfrom
ROU-12319

Conversation

@JoaoFerreira-FrontEnd

@JoaoFerreira-FrontEnd JoaoFerreira-FrontEnd commented Nov 28, 2025

Copy link
Copy Markdown
Contributor

Issue number: internal


What is the current behavior?

  • When Avatar has an icon as content and is disabled, the icon is above the disabled layer.

What is the new behavior?

  • add avatar with icon on disabled state page;
  • add avatar with icon on snapshots tests;
  • change disable state to be the :after pseudo element avatar content;

Does this introduce a breaking change?

  • Yes
  • No

Other information

state

- add avatar with icon on snapshots tests;
- change disable state to be :after avatar content;
@JoaoFerreira-FrontEnd JoaoFerreira-FrontEnd requested a review from a team as a code owner November 28, 2025 18:32
@JoaoFerreira-FrontEnd JoaoFerreira-FrontEnd added the package: core @ionic/core package label Nov 28, 2025
@JoaoFerreira-FrontEnd JoaoFerreira-FrontEnd added the type: bug a confirmed bug report label Nov 28, 2025
@vercel

vercel Bot commented Nov 28, 2025

Copy link
Copy Markdown

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

Project Deployment Preview Comments Updated (UTC)
ionic-framework Ready Ready Preview Comment Dec 3, 2025 2:07pm

@brandyscarney brandyscarney 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!

@JoaoFerreira-FrontEnd JoaoFerreira-FrontEnd merged commit 10f4197 into next Dec 3, 2025
49 checks passed
@JoaoFerreira-FrontEnd JoaoFerreira-FrontEnd deleted the ROU-12319 branch December 3, 2025 14:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

package: core @ionic/core package type: bug a confirmed bug report

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants