Skip to content
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

window client side decorations problems #1522

Closed
totaam opened this issue May 21, 2017 · 17 comments
Closed

window client side decorations problems #1522

totaam opened this issue May 21, 2017 · 17 comments

Comments

@totaam
Copy link
Collaborator

totaam commented May 21, 2017

Issue migrated from trac ticket # 1522

component: html5 | priority: major | resolution: fixed

2017-05-21 18:07:29: Denis01 created the issue


Hello,
faced some problems with the beta version 2.1-15785 64-bit (centos7.3, HTML client)
In the attached files the application look in 2.0.2-15657 64-bit version (2_0_2_version.jpg) and in 2.1-15785 (2_1_version.jpg).
So in 2.1 there is no "band" above application (marked with string) but in 2.0.2 there is band. So picking the application on this band by mouse pointer in 2.0.2 version i can easily move and resize the application window but in 2.1 version application sceen either frozen or jumping in unpredicted way.
What it could be? Could you help!

On other hand (probably it is another situation and just for information as it could be normal way of working) tried to avoid this problem installed Proxy server 2.1-15785 beta and Target server with application behind with 2.0.2-15657. And Proxy of higher version can't connect to Target server with lower version.
The error:

2017-05-21 12:35:28,587 DotXpra(/var/run/user/1000/xpra, ['/var/run/user/1000/xpra'] - 1000:1000 - 111).socket_path(:proxy-2578)=/var/run/user/1000/xpra/localhost.localdomain-proxy-2578
2017-05-21 12:35:28,587 create_control_socket: socket path='/var/run/user/1000/xpra/localhost.localdomain-proxy-2578', uid=1000, gid=1000, state=DEAD
2017-05-21 12:35:28,587 Warning: failed to create socket directory '/var/run/user/1000/xpra'
2017-05-21 12:35:28,587  [Errno 2] No such file or directory: '/var/run/user/1000/xpra'
2017-05-21 12:35:28,587 create_unix_domain_socket failed for '/var/run/user/1000/xpra/localhost.localdomain-proxy-2578'
Traceback (most recent call last):
  File "/usr/lib64/python2.7/site-packages/xpra/server/proxy/proxy_instance_process.py", line 273, in create_control_socket
    sock, self.control_socket_cleanup = create_unix_domain_socket(sockpath, None, 0o600)
  File "/usr/lib64/python2.7/site-packages/xpra/scripts/server.py", line 376, in create_unix_domain_socket
    listener.bind(sockpath)
  File "/usr/lib64/python2.7/socket.py", line 224, in meth
    return getattr(self._sock,name)(*args)
error: [Errno 2] No such file or directory
2017-05-21 12:35:28,612 Error: failed to setup control socket '/var/run/user/1000/xpra/localhost.localdomain-proxy-2578':
2017-05-21 12:35:28,612  [Errno 2] No such file or directory
@totaam
Copy link
Collaborator Author

totaam commented May 21, 2017

2017-05-21 18:07:52: Denis01 uploaded file 2_0_2_version.jpg (43.6 KiB)

2_0_2_version.jpg

@totaam
Copy link
Collaborator Author

totaam commented May 21, 2017

2017-05-21 18:08:01: Denis01 uploaded file 2_1_version.jpg (49.3 KiB)

2_1_version.jpg

@totaam
Copy link
Collaborator Author

totaam commented May 21, 2017

2017-05-21 18:28:53: Denis01 commented


So to make it simple - without this "band" (in 2.1 beta) just can't use the application. Screen is jumping, can't click on buttons...

@totaam
Copy link
Collaborator Author

totaam commented May 22, 2017

2017-05-22 07:23:15: Denis01 commented


Morning!
Tested with some other applications.
Quite the same situation in case of 2.1 version without "band" (or bar i dont't know how to call it correctly) above the application)
Some applications work, others - no.
For those which don't work: sometimes can pull/move normally the sceeen inside the browser but generally buttons dont't work(no reaction upon the click on the buttons).

@totaam
Copy link
Collaborator Author

totaam commented May 22, 2017

2017-05-22 07:24:28: Denis01 changed component from android to html5

@totaam
Copy link
Collaborator Author

totaam commented May 22, 2017

2017-05-22 10:37:29: antoine changed owner from antoine to Denis01

@totaam
Copy link
Collaborator Author

totaam commented May 22, 2017

2017-05-22 10:37:29: antoine changed title from Some issues with beta version to window client side decorations problems

@totaam
Copy link
Collaborator Author

totaam commented May 22, 2017

2017-05-22 10:37:29: antoine commented


What you call "band" is known as the "window title bar".
2.1 improved support for window attributes set by the applications, including the ability for applications to request the window to not be decorated (removing the title bar): see #1491 for details.

From the screenshot, it looks like the application is drawing its own title bar (aka "client side decorations") and therefore it makes sense for us to not draw another one on top. It looks like this is working as expected.

Using the proxy should not make any difference at all here.

If however, the window does not behave as expected, for example if it is difficult to trigger a move or resize, then that means we're not handling the application requests properly.
In which case, please specify the browser used and an application I can use to reproduce the problem.

The create_unix_domain_socket failed is completely unrelated and does not belong in this ticket.

@totaam
Copy link
Collaborator Author

totaam commented May 22, 2017

2017-05-22 10:53:12: Denis01 commented


Hello,
Thanks for the detailed explanations!
Browser Chrome 53.0.2785
And the application - the distrubutives are about 0,5 Gb. How can i pass them? Or i can set up a test server to connect and check the situation directly on it.

@totaam
Copy link
Collaborator Author

totaam commented May 22, 2017

2017-05-22 10:56:02: antoine commented


Please find an open-source application that reproduces the problem.
I need the source, and I need to be able to run locally to debug.
This test application works more or less correctly: [/browser/xpra/trunk/src/tests/xpra/test_apps/test_window_initiatemoveresize.py test_window_initiatemoveresize]

@totaam
Copy link
Collaborator Author

totaam commented May 22, 2017

2017-05-22 12:13:41: Denis01 commented


Ok, will try to find smth.
But would it be possible (for a time being, awating me searching) to have some setting in 2.1 version with activation and disactivation. Like "window decoration" = ON/OFF? As it could appear with one or another application because of various reasons...

@totaam
Copy link
Collaborator Author

totaam commented May 22, 2017

2017-05-22 15:22:54: antoine commented


have some setting in 2.1 version with activation and disactivation
I don't have time for this, but I will take a patch if you submit one.

@totaam
Copy link
Collaborator Author

totaam commented May 23, 2017

2017-05-23 12:41:43: Denis01 commented


Hello,
have a look on #1491 - won't be an easy task for me :-) but will do the best.
Seems that in:

  • r15527
  • r15517
    have the modifications for the "decoration" settings.
    But even if the version Centos7.3 2.1-15785 is installed don't see (for example)
if ("title" in metadata) {

in Window.js file and not

if ("decorations" in metadata) {

as described in r15517
So probably the version is not completely assembled.
Could you please publish new beta release with all the modifs just to be sure everything is in newest version.
P.S. And for a while i am keeping the fingers crossed - the newest version of files might resolve the problem....

@totaam
Copy link
Collaborator Author

totaam commented May 23, 2017

2017-05-23 14:15:56: Denis01 commented


Apologize for stupid remark in "comment:9"
seems that have found everything

@totaam
Copy link
Collaborator Author

totaam commented May 23, 2017

2017-05-23 18:35:11: Denis01 commented


Think this ticket should be closed.
Will create new one for evolution.
Settings to choose decoration type and assign to myself - will prepare the script

@totaam
Copy link
Collaborator Author

totaam commented May 23, 2017

2017-05-23 18:35:34: Denis01 changed status from new to closed

@totaam
Copy link
Collaborator Author

totaam commented May 23, 2017

2017-05-23 18:35:34: Denis01 set resolution to fixed

@totaam totaam closed this as completed May 23, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant