chore: move attachments link to step body

[Skn0tt]

Follow-up to #34037, as discussed in our review.

Before:

After:

(don't be spooked by the duplication in the screenshot, it's just to show how labels are spaced)

Copilot reviewed 2 out of 3 changed files in this pull request and generated no comments.

Files not reviewed (1)

• packages/html-reporter/src/links.css: Language not supported

[github-actions]

Test results for "tests 1"

1 failed
❌ [playwright-test] › runner.spec.ts:118:5 › should ignore subprocess creation error because of SIGINT @macos-latest-node18-1

5 flaky

⚠️ [firefox-page] › page/page-evaluate.spec.ts:403:3 › should throw for too deep reference chain @firefox-ubuntu-22.04-node18
⚠️ [webkit-library] › library/browsertype-connect.spec.ts:356:5 › launchServer › should reject navigation when browser closes @webkit-ubuntu-22.04-node18
⚠️ [webkit-library] › library/screenshot.spec.ts:219:14 › element screenshot › element screenshot should work with device scale factor @webkit-ubuntu-22.04-node18
⚠️ [webkit-library] › library/tracing.spec.ts:209:5 › should respect tracesDir and name @webkit-ubuntu-22.04-node18
⚠️ [playwright-test] › ui-mode-test-watch.spec.ts:145:5 › should watch all @windows-latest-node18-1

37499 passed, 650 skipped
✔️✔️✔️

Merge workflow run.