playwright icon indicating copy to clipboard operation
playwright copied to clipboard

[Bug]: Current changes in main broke the Trace Viewer

Open Snooz82 opened this issue 1 year ago • 3 comments

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: Image

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 Image

Image

Additional context

No response

Environment

irrelevant

Snooz82 avatar Oct 18 '24 16:10 Snooz82

Ping @pavelfeldman 😉

Snooz82 avatar Oct 18 '24 17:10 Snooz82

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.

pavelfeldman avatar Oct 18 '24 19:10 pavelfeldman

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.

Snooz82 avatar Oct 19 '24 22:10 Snooz82

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!

mxschmitt avatar Oct 21 '24 08:10 mxschmitt