[Bug]: Current changes in main broke the Trace Viewer
Version
Development
Steps to reproduce
Open Trace viewer that is create by the tests of tests/library/trace-viewer.spec.ts
I am currently implementing groups for tracing.
And the actions tree in trace viewer has been changed from <div role="listitem" class="list-view-entry"> to <div role="treeitem" aria-selected="false"><div class="tree-view-entry">.
That does cause a wrong rendering.
Expected behavior
This is how it should look like:
i think the changed div with the role treeitem also shall have the class tree-view-entry. Not the child.
Actual behavior
This is how it looks
Additional context
No response
Environment
irrelevant
Ping @pavelfeldman 😉
Hey, thanks for the heads up. I'm reworking the list into tree for better a11y here and I think flex: none is missing on the treeitem. I have a large change that should include this fix along the way: https://github.com/microsoft/playwright/pull/33169.
yea. saw the changes, because my tests for PR #33081 were failing after rebase.
i saw that it seems to be caused by your last changes to the action tree. thats why i pinged you.
The fix should be available on NPM and available on https://trace.playwright.dev/next/.
Please re-file / ping me if you want to re-open!