Skip to content

fix(material/sort): double focus indicators when strong focus is enabled - #33520

Merged
crisbeto merged 1 commit into
angular:mainfrom
crisbeto:33519/sort-header-dual-indicator
Jul 13, 2026
Merged

fix(material/sort): double focus indicators when strong focus is enabled#33520
crisbeto merged 1 commit into
angular:mainfrom
crisbeto:33519/sort-header-dual-indicator

Conversation

@crisbeto

@crisbetocrisbeto commented Jul 11, 2026

Copy link
Copy Markdown
Member

Fixes that the sort header both its usual focus indicator and the strong focus one when strong focus is enabled.

Fixes#33519.

@crisbeto
fix(material/sort): double focus indicators when strong focus is enabled
1ba7bdc
Fixes that the sort header both its usual focus indicator and the strong focus one when strong focus is enabled.
Fixesangular#33519.
@crisbetocrisbeto added the target: patch This PR is targeted for the next patch release label Jul 11, 2026
@crisbeto
crisbeto requested a review from ok7saiJuly 12, 2026 16:52
@crisbetocrisbeto added the action: merge The PR is ready for merge by the caretaker label Jul 13, 2026
@crisbeto
crisbeto merged commit 14d896a into angular:mainJul 13, 2026
33 of 36 checks passed
@crisbeto

crisbeto commented Jul 13, 2026

Copy link
Copy Markdown
MemberAuthor

This PR was merged into the repository. The changes were merged into the following branches:

crisbeto added a commit that referenced this pull request Jul 13, 2026
@crisbeto
fix(material/sort): double focus indicators when strong focus is enab…
092b55a
…led (#33520)
Fixes that the sort header both its usual focus indicator and the strong focus one when strong focus is enabled.
Fixes#33519.
(cherry picked from commit 14d896a)
@angular-automatic-lock-bot

angular-automatic-lock-botBot commented Aug 13, 2026

Copy link
Copy Markdown

This pull request has been automatically locked due to inactivity.
Please file a new issue if you are encountering a similar or related problem.

Read more about our automatic conversation locking policy.

This action has been performed automatically by a bot.

@angular-automatic-lock-botangular-automatic-lock-botBot locked and limited conversation to collaborators Aug 13, 2026
Sign up for freeto subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

@ok7saiok7saiok7sai approved these changes
@andrewseguinandrewseguinAwaiting requested review from andrewseguin
@tjshiutjshiuAwaiting requested review from tjshiu

Assignees

No one assigned

Labels

action: mergeThe PR is ready for merge by the caretakerarea: material/sorttarget: patchThis PR is targeted for the next patch release

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

bug(sort header): Default focus state do not work well with strong focus indicator

2 participants

@crisbeto@ok7sai