-
-
Notifications
You must be signed in to change notification settings - Fork 53
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. Weβll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Feat/create project desc highlight #2001
Conversation
β¦ng element hover
β¦ be focused when it's corresponding element is hovered
@spwoodcock can't debug why the test cases are failing. When the test cases run is the Debug true or false? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Comment, otherwise looks good!
@@ -58,7 +58,7 @@ services: | |||
# timeout: 5s | |||
|
|||
ui-test: | |||
image: "mcr.microsoft.com/playwright:${PLAYWRIGHT_TAG:-v1.48.1}" | |||
image: "mcr.microsoft.com/playwright:${PLAYWRIGHT_TAG:-v1.44.1}" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm not sure downgrading this version is a good idea.
It might cause inconsistencies with the version used during CI too - they always have to match else browser versions used for testing will differ
For others to know, we discussed this bilaterally and decided to address the test issues in the new year |
for more information, see https://pre-commit.ci
What type of PR is this? (check all applicable)
Related Issue
Describe this PR
This PR contains the following works:
Descripition
componentScreenshots
Screencast.from.18-12-24.09.33.35.+0545.webm