Skip to content

fix(material/slider): not picking up static direction#33006

Merged
crisbeto merged 1 commit intoangular:mainfrom
crisbeto:32970/slider-direction
Mar 30, 2026
Merged

fix(material/slider): not picking up static direction#33006
crisbeto merged 1 commit intoangular:mainfrom
crisbeto:32970/slider-direction

Conversation

@crisbeto
Copy link
Copy Markdown
Member

The slider was reading its direction once on init and one when it changes, however the initial read was too early which meant that the dir directive might not have received its value yet.

Fixes #32970.

The slider was reading its direction once on init and one when it changes, however the initial read was too early which meant that the `dir` directive might not have received its value yet.

Fixes angular#32970.
@crisbeto crisbeto added the target: patch This PR is targeted for the next patch release label Mar 28, 2026
@pullapprove pullapprove Bot requested review from andrewseguin and tjshiu March 28, 2026 08:22
@crisbeto crisbeto added the action: merge The PR is ready for merge by the caretaker label Mar 30, 2026
@crisbeto crisbeto merged commit a7d5561 into angular:main Mar 30, 2026
30 of 32 checks passed
@crisbeto
Copy link
Copy Markdown
Member Author

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

crisbeto added a commit that referenced this pull request Mar 30, 2026
The slider was reading its direction once on init and one when it changes, however the initial read was too early which meant that the `dir` directive might not have received its value yet.

Fixes #32970.

(cherry picked from commit a7d5561)
@angular-automatic-lock-bot
Copy link
Copy Markdown

This issue 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-bot angular-automatic-lock-bot Bot locked and limited conversation to collaborators Apr 30, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

action: merge The PR is ready for merge by the caretaker area: material/slider target: patch This PR is targeted for the next patch release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

bug(Slider): direction rtl doesn't work properly

2 participants