-
-
Notifications
You must be signed in to change notification settings - Fork 44
Assure Update window width does not break OpenQA #248
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
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #248 +/- ##
==========================================
- Coverage 93.18% 93.16% -0.02%
==========================================
Files 58 58
Lines 11065 11065
==========================================
- Hits 10311 10309 -2
- Misses 754 756 +2 ☔ View full report in Codecov by Sentry. |
OpenQA test summaryComplete test suite and dependencies: https://openqa.qubes-os.org/tests/overview?distri=qubesos&version=4.3&build=2025031404-4.3&flavor=pull-requests Test run included the following:
New failures, excluding unstableCompared to: https://openqa.qubes-os.org/tests/overview?distri=qubesos&version=4.3&build=2025021804-4.3&flavor=update
Failed tests25 failures
Fixed failuresCompared to: https://openqa.qubes-os.org/tests/129058#dependencies 10 fixed
Unstable testsPerformance TestsPerformance degradation:32 performance degradations
Remaining performance tests:40 tests
|
398add7 to
1af77ee
Compare
It should be the window border. I removed further 12 pixels. This should work. |
|
Looks like it's broken still/again: https://openqa.qubes-os.org/tests/133217#step/qui_widgets_update/17 |
Is this a multi-display system? With the main one over 1024x768? most probably not. It appears it happens on both Onigiri and hal9007 :/ |
|
No, it's a single 1024x768 |
Refrence: #246 (comment)