Feature Request Tracker item #3486983, was opened at 2012-02-12 05:45
Message generated for change (Comment added) made by clohr
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=1126849&aid=3486983&group_id=254363

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: FLTK viewer
Group: None
Status: Open
Resolution: None
Priority: 5
Private: No
Submitted By: clohr (clohr)
Assigned to: Pierre Ossman (ossman_)
Summary: A dialog box to send DesktopSize requests

Initial Comment:
Hi,
  I propose a dialog box to vncviewer allowing user to send DesktopSize 
requests (see attached patch).
I hope you find this usefull ;-)

Regard.


----------------------------------------------------------------------

>Comment By: clohr (clohr)
Date: 2012-02-29 12:59

Message:
Hello,
Even if my proposal has not it been selected as-is, I'm glad to see that
some people are interested.
My initial need was to be able to send DeskTopSize requests during the
session, not just when starting the viewer.
I think both approaches (dialog box and viewer-window resizing) are
complementary and are not opposed. But one may be sufficient.

However we must keep in mind that server-size of viewer-size are two
different things. 
One can choose to:
- Keep them synchronized: When the server-size changes, viewers resize
their window. And when a user resizes a viewer, it sends a DesktopSize
request to the server that resize itself.
- Do not keep them synchronized: The viewer uses either scrollbars or
scaling to display server's pictures in its window.


In fact, I have also seen strange reactions when the server receives
DeskTopSize requests with non-standard sizes. One criterion is that the
width must be a multiple of 8. But there may be other criteria that I was
not able to determine. We need an expert for this part. :-)

Best regards

----------------------------------------------------------------------

Comment By: Arthur Huillet (ahuillet)
Date: 2012-02-29 01:22

Message:
Hello,
Pierre Ossman didn't answer you on this ticket but we had a talk on IRC. He
stated that he disagreed rather strongly with a dialog box like your patch
does, and that he favors the approach I implemented where the desktop is
resized when the user resizes the viewer window.

Now, I've tested the patch you have a  little, and in some cases it doesn't
work - I suspect that the DesktopSize request is not sent at the right
time, and that we should not wait for the next FB update. If you try my
patch you'll see what I'm talking about - in many cases the desktop is
*not* resized when it should.

----------------------------------------------------------------------

Comment By: clohr (clohr)
Date: 2012-02-15 09:44

Message:
Many thanks ahuillet
I added your patch (see file resize_desktop.patch)

Regards

----------------------------------------------------------------------

Comment By: clohr (clohr)
Date: 2012-02-15 07:39

Message:
Hello,
Many thanks for your remarks.
Here's a new version of my patch with the misspelling corrected.
(see file tigervnc-1.1.90_DesktopSizeDialog_v2.patch.gz)

Note: If you want, send me your file by email and I'll add it to this
ticket.

Regards

----------------------------------------------------------------------

Comment By: Arthur Huillet (ahuillet)
Date: 2012-02-15 05:05

Message:
Oh, apparently I can't attach a file to a ticket that isn't mine. 

----------------------------------------------------------------------

Comment By: Arthur Huillet (ahuillet)
Date: 2012-02-15 05:05

Message:
Hello,

I have improved your patch a little bit by adding the ability to simply
resize the viewer's window to change the desktop size. Please see the
attached file.

----------------------------------------------------------------------

Comment By: Arthur Huillet (ahuillet)
Date: 2012-02-14 02:30

Message:
Hello,

I am *not* a TigerVNC developer so please take my comments with a grain of
salt: they are quite probably not relevant.

1- I confirm that your patch works.
2-spelling: s/traditionnal/traditional/

----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=1126849&aid=3486983&group_id=254363

------------------------------------------------------------------------------
Virtualization & Cloud Management Using Capacity Planning
Cloud computing makes use of virtualization - but cloud computing 
also focuses on allowing computing to be delivered as a service.
http://www.accelacomm.com/jaw/sfnl/114/51521223/
_______________________________________________
Tigervnc-devel mailing list
Tigervnc-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/tigervnc-devel

Reply via email to