Skip to content

docs: convert list/pag/progb/progs/rad/rip/elev to signals#33497

Merged
crisbeto merged 1 commit into
angular:mainfrom
msmallest:docs-opzl-list-pag-progb-progs-rad-rip-ele
Jul 7, 2026
Merged

docs: convert list/pag/progb/progs/rad/rip/elev to signals#33497
crisbeto merged 1 commit into
angular:mainfrom
msmallest:docs-opzl-list-pag-progb-progs-rad-rip-ele

Conversation

@msmallest

Copy link
Copy Markdown
Contributor

Description

Converts the following components to use signals

Part of an effort to address this issue about compatibility with new defaults: #33443

  • List
  • Paginator
  • Progress Bar
  • Progress Spinner
  • Radio Button
  • Ripple (core) example
  • Elevation (see note)

Ripple and Elevation are considered "Core" items, but have no "Examples" section in the "Core" page. However, Ripples are in the docs Components sidenav and the "Examples" tab houses the example that was updated.

And I also converted a (core) elevation example. Unlike Ripple which was a "Components" sidenav item whose example is nested under the /core folder of the component examples, I could not find usage of Elevation in practice in the docs. So I was not able to smoketest it, but it should be legit as I double checked it when reviewing my code first.

PR scope

I thought that 7 components worth of changes was a good chunk. There is more chunks to come for other components.

This time around I only made this one branch, since the last PR as one chunk worked well: #33492. Thank you.

Extras

For personal reference, here is my checklist of doc examples worked through: https://gist.github.com/msmallest/f77adb004b9e88c49b3a248bab90a26a?permalink_comment_id=6237101#gistcomment-6237101

@pullapprove pullapprove Bot requested review from ok7sai and tjshiu July 6, 2026 16:28
@angular-robot angular-robot Bot added the area: docs Related to the documentation label Jul 6, 2026
@crisbeto crisbeto added action: merge The PR is ready for merge by the caretaker target: patch This PR is targeted for the next patch release labels Jul 7, 2026
@crisbeto crisbeto merged commit 8205e24 into angular:main Jul 7, 2026
34 of 36 checks passed
@crisbeto

crisbeto commented Jul 7, 2026

Copy link
Copy Markdown
Member

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

crisbeto pushed a commit that referenced this pull request Jul 7, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

action: merge The PR is ready for merge by the caretaker area: docs Related to the documentation target: patch This PR is targeted for the next patch release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants