Skip to content

fix(trace-viewer): label filter inputs and make error source link keyboard-accessible - #42465

Closed
Ashraf Ali (ashrafiucse) wants to merge 2 commits into
microsoft:mainfrom
ashrafiucse:fix-42463
Closed

fix(trace-viewer): label filter inputs and make error source link keyboard-accessible#42465
Ashraf Ali (ashrafiucse) wants to merge 2 commits into
microsoft:mainfrom
ashrafiucse:fix-42463

Conversation

@ashrafiucse

Copy link
Copy Markdown

Summary

  • Add aria-label to the network tab filter input and the UI mode filter input, matching the convention used elsewhere (Filter actions, Filter steps, Search tests)
  • Convert the error source location in the Errors tab from a click-only <span> to a real <button> so it is focusable and keyboard-activatable, styled identically to before

Fixes #42463

In extension mode, test-run connections engage the reuse-browsers mode,
which deliberately keeps pages alive across connections ("Don't close
the pages so that user could debug them"). However, the shared browser
was not marked as shared, so the connection cleanup treated the
contexts created by the connection as isolated and closed them all on
disconnect ("Global context cleanup"), killing the page being debugged.

Mark the browser as shared in reuse-browsers mode, so contexts with
pages survive the connection drop and are picked up by the next
connection. Empty contexts are still cleaned up on disconnect as
before.

This restores the pre-1.54 behavior where stopping a debug session in
the VS Code extension keeps the browser open for inspection and
recording.

Fixes: microsoft#37822
…board-accessible

Filter inputs in the network tab and UI mode relied on placeholder alone,
which is not a reliable accessible name. The error source location in the
Errors tab was a span with a click handler, so it could not be focused or
activated from the keyboard - now a real button styled identically.

Fixes: microsoft#42463
@dgozman

Copy link
Copy Markdown
Collaborator

Closing in favor of #42469.

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.

[Bug]: two trace viewer filter inputs have no accessible name, and the error source link is mouse-only

2 participants