Skip to content

Link <Index> in list rendering overview sentence#1529

Closed
Copilot wants to merge 12 commits into
mainfrom
copilot/fix-review-comment-3329999996
Closed

Link <Index> in list rendering overview sentence#1529
Copilot wants to merge 12 commits into
mainfrom
copilot/fix-review-comment-3329999996

Conversation

Copilot AI commented May 31, 2026

Copy link
Copy Markdown
Contributor

This PR addresses the review feedback on PR #1527 to update only the referenced comment location. The issue was that the list-rendering intro sentence mentioned both <For> and <Index>, but only one component was linked.

  • Scope

    • Updated a single line in src/routes/(0)concepts/(1)control-flow/(1)list-rendering.mdx.
    • No other docs/pages/components were modified.
  • Documentation correction

    • Added the missing <Index> reference link to the existing sentence that already linked <For>.
    • Used the canonical route: /reference/components/index-component.
  • Applied change (exact)

    When dealing with dynamic data, Solid offers two ways to render lists: the [`<For>`](/reference/components/for) and [`<Index>`](/reference/components/index-component) components.

@bolt-new-by-stackblitz

Copy link
Copy Markdown

Review PR in StackBlitz Codeflow Run & review this pull request in StackBlitz Codeflow.

@netlify

netlify Bot commented May 31, 2026

Copy link
Copy Markdown

Deploy Preview for solid-docs ready!

Name Link
🔨 Latest commit bdf5503
🔍 Latest deploy log https://app.netlify.com/projects/solid-docs/deploys/6a3621f6ace5eb0008e1be14
😎 Deploy Preview https://deploy-preview-1529--solid-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
🤖 Make changes Run an agent on this branch

To edit notification comments on pull requests, go to your Netlify project configuration.

Copilot AI changed the title [WIP] Fix code according to review comment 3329999996 Link <Index> in list rendering overview sentence May 31, 2026
Copilot AI requested a review from LadyBluenotes May 31, 2026 19:53
@LadyBluenotes LadyBluenotes marked this pull request as ready for review June 14, 2026 01:49
@LadyBluenotes LadyBluenotes enabled auto-merge (squash) June 20, 2026 05:10
auto-merge was automatically disabled June 20, 2026 05:15

Pull request was closed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants