Opened 15 months ago
Last modified 3 weeks ago
#2455 assigned defect
xpra-shadow new borders and window size miscalculations
Reported by: | stdedos | Owned by: | Antoine Martin |
---|---|---|---|
Priority: | critical | Milestone: | 4.2 |
Component: | client | Version: | 3.0.x |
Keywords: | Cc: |
Description (last modified by )
After changing to the "new window styles" (between r24005 and r24195), I noticed that the shadow works "a bit differently"
Please see attached images.
- Image may not be fully initialized
- Viewport calculation is wrong
- On fullsize, viewport is correct, but other monitors "seep in"
I am not complaining so much about (3), but, if client could scale shadow window to perfectly fit the client's available viewport for maximized windows (i.e. window + the Start bar), then this would bring unnecessary drawing outside of the screen.
I don't know if client would be able to do that with Shadow windows (i.e. make the scaling fill pixel-perfectly the whole screen), but it would be a nice feature (reported in #2458).
Attachments (4)
Change History (20)
Changed 15 months ago by
Attachment: | Xpra_cmd_2019-10-19_12-28-34-1.png added |
---|
Changed 15 months ago by
Attachment: | Xpra_cmd_2019-10-19_12-30-04-2.png added |
---|
comment:2 Changed 15 months ago by
"New window style" updates were done on r24164? Intentionally or Unintentionally?
comment:3 follow-up: 4 Changed 15 months ago by
Owner: | changed from Antoine Martin to stdedos |
---|
- I'm not really sure what to look for on those screenshots
Image may not be fully initialized
Is the grey area the xpra client showing a shadow server?
How long does it stay like that?
Viewport calculation is wrong
Too big or too small?
xpra info
output would show us what it is trying to do.
On fullsize, viewport is correct, but other monitors "seep in"
"viewport" - are those monitors?
How many monitors? How much do they "seep in" by?
but, if client could scale shadow window to perfectly fit the client's available viewport for maximized windows
..
I don't know if client would be able to do that with Shadow windows (i.e. make the scaling fill pixel-perfectly the whole screen), but it would be a nice feature.
This would be a different feature request (we may have a ticket already)
"New window style" updates were done on r24164? Intentionally or Unintentionally?
Are you sure that this changeset does something? How did you identify it?
As far as I can tell, it just moves things around and merges two modules.
There is only one opengl backend supported in v4, the "native" one.
comment:4 Changed 15 months ago by
Replying to Antoine Martin:
- I'm not really sure what to look for on those screenshots
Xpra_cmd_2019-10-19_12-25-26.png
Full sized window being partially drawn
Xpra_cmd_2019-10-19_12-28-34-1.png
Maximized window being fully drawn, but missing screen image from below (need to compare with the next one)
Xpra_cmd_2019-10-19_12-30-04-2.png
Full screen window containing the missing portion of the screen (from above), and the next screen (DP-4) being "glued" together with DP-2
Image may not be fully initialized
Is the grey area the xpra client showing a shadow server?
Yes
How long does it stay like that?
Until "some change" is made - manipulating the size of the xpra client window. Maybe when the screen is changed (new draws / mouse movement in that area), I haven't verified that one.
Viewport calculation is wrong
Too big or too small?
xpra info
output would show us what it is trying to do.
"Pushed down" from the new style of the window toolbar. You've added e.g. 100px above, the window needs to be "extended" downwards for that extra 100px.
On fullsize, viewport is correct, but other monitors "seep in"
"viewport" - are those monitors?
How many monitors? How much do they "seep in" by?
3 monitors in total - from left to right DP-{0,2,4}.
They don't hide anything from the existing screen (i.e. DP-2), but they simply "manifest" on full screen
but, if client could scale shadow window to perfectly fit the client's available viewport for maximized windows
..
I don't know if client would be able to do that with Shadow windows (i.e. make the scaling fill pixel-perfectly the whole screen), but it would be a nice feature.
This would be a different feature request (we may have a ticket already)
Have another one :-D #2458
"New window style" updates were done on r24164? Intentionally or Unintentionally?
Are you sure that this changeset does something? How did you identify it?
As far as I can tell, it just moves things around and merges two modules.
There is only one opengl backend supported in v4, the "native" one.
I am not sure, I have gone through all patchsets from r24005 to r24195 manually. The only thing that could have possible looked like being "close" to this one was that. Of course, I can be wrong.
Changed 15 months ago by
Attachment: | Xpra_cmd_2019-10-19_12-25-26.png added |
---|
Changed 15 months ago by
Attachment: | xpra-info-2455.log added |
---|
comment:5 Changed 15 months ago by
"Everything seems to be as it should be" (from xpra info 0 | grep -P '1920|2560'
), but I cannot be sure.
That new style window toolbar does not appear to be windows-native, and I am not sure how much Windows likes that or not.
comment:6 Changed 15 months ago by
That new style window toolbar does not appear to be windows-native, and I am not sure how much Windows likes that or not.
Are you sure you're not comparing GTK2 with GTK3 builds?
comment:7 Changed 15 months ago by
This is how I look for new Windows versions for a long time:
https://www.xpra.org/beta/windows/?C=M&O=D&P=*ython3*r24*.zip*
(obviously I change r24
accordingly)
comment:8 Changed 15 months ago by
This is how I look for new Windows versions for a long time:
..
All downloads currently in the beta area are based on GTK3.
Right around the time of what you call "windows-native", the default was changed from GTK2 to GTK3.
The older Python2 builds were simply named "Xpra*". (though, going forward, the 3.0 builds will now have "Python2" or "Python3" to make it easier to distinguish)
comment:9 Changed 15 months ago by
So, what I mean is this: if the bug occurs with 3.0, please try both the latest python2 and python3 builds so we know if the bug comes from the switch to gtk3 (python3).
comment:10 Changed 15 months ago by
comment:11 Changed 15 months ago by
If there is a py3-gtk3 and py2-gtk2 build somewhere, please provide the links and I'll retry testing.
The beta area: https://xpra.org/beta/windows/
Has both python2 and python3 builds of 3.0.1RC.
The stable area https://xpra.org/dists/windows/
Also has both python2 (unprefixed) and python3 builds.
comment:12 Changed 15 months ago by
Owner: | changed from stdedos to Antoine Martin |
---|
https://xpra.org/beta/windows/Xpra-Python2-x86_64_3.0.1-r24232.zip does not exhibit such issue.
Drawing, contents, etc - everything is complete and as expected.
comment:13 Changed 10 months ago by
Milestone: | 4.0 → 4.1 |
---|---|
Owner: | changed from Antoine Martin to Antoine Martin |
Status: | new → assigned |
See also #530
comment:14 Changed 10 months ago by
Description: | modified (diff) |
---|
comment:15 Changed 8 months ago by
Priority: | major → critical |
---|
comment:16 Changed 3 weeks ago by
Milestone: | 4.1 → 4.2 |
---|
client: