Created attachment 55084 [details] Patch for making the size dependent on the view window size Version: unspecified (using KDE 4.4.5) OS: Linux When opening an RDP connection it always does this in the predefined resolutions: 800x600, 1024x768, ... While I have seen that this should get solved in the future by freerdp, I wrote a small patch that does exactly this. Reproducible: Always This patch adds a general config option. Do use it together with "Always show tab bar". Otherwise you will get that the first rdp connection will get a slightly bigger resolution and as soon as you open the second connection you will get scrollbars for the first connection.
Any idea if this will be added in the future release? I think it is a very small patch and make krdc even more productive.
Do I get it right that your patch takes the size of the window when the RDP connection gets opened? It does not resize anymore when you resize the KRDC window?! How will freerdp handle that? Anyway, we cannot apply such a patch before trunk is open for KDE SC 4.7 features.
I have no idea how freerdp will handle that :p there is not alot of information about this so I might have understand it wrong :p. From what I understand you cannot resize an rdp connection. So when you resize the Krdc window (let me think of a reason why and when I do this.. never actually, I want the view as big as possible; Anyway, I made a configuration option for people who don't like it ;) ) If I ever resize then I just double click the link again and it reconnects and uses the new view size (so my programs are still running). It would however be possible to make krdc reconnect automatically. Anyway, using the standard 1024x768, ... resolutions the same issue arises when you resize the window to small... so that shouldn't keep this feature from being added.
Okay, thanks for your patch. One question: why don't you add a new combobox item instead of the checkbox? I think it would fit into the combobox just as "Use current desktop resolution".
The item for combobox would only make this change for 1 connection while I use krdc all the time, so I want it as a default for each connection(as I like full usage of the available screen ;) ).
I found the option "Resize to fit" in krdc 4.6.1. Is this the functionality discussed in this "ticket"? If it is, it does not work for me. Just checking (I modified my patch to continue working).
Is there any chance my fix/feature will ever be include in the source code? Or is there any chance that the "Resize to fit" will ever work? I am on 4.6.3 now and still I have to apply this patch myself :S It is resolved, it is a few lines and I don't see why anybody would be afraid of including it in the sources. (unless you like alot of wasted space on your screen of course).
It's not fixed yet, reopen.
Please let me know what is still not working so I can make a patch to get this closed and included. "It is not fixed yet, reopen" doesn't say alot to me. There is a diff in attachment. It probably need minor tweeking to get to work against the current working tree.
deadeyes: I'm sorry about the delay of resolving this issue. Thanks for your work anyway! I really appreciate any patches for bug reports / wishes. Since it introduces new user visible text-messages, it needs to wait until trunk is open again (i.e. 4.8). I will try to apply this patch and test it.
So I created another patch.. This time I added an entry to the Resolution dropdown list. It sets the width and height to 0 (AUTOSIZE). When the rdp view is opened it uses the size of the current tab space. When will this finally be included? I don't care how but it would be great after 2 years that this finally is included in the sources.
Created attachment 69517 [details] Make rdp fit to the whole canvas instead of a predetermined size
Created attachment 75980 [details] Fit RDP screen to KRDC size Good news! I have looked into this patch and slighty modified it. I will commit it after the port from rdesktop to freerdp which will be included in the KDE SC 4.11 release. @deadeyes: Thanks a lot for your patch! :) Sorry, it took quite some time until I had found time to look int it.
SVN commit 1330137 by uwolfer: Add setting to size RDP view fitting into current KRDC size. Work is based on a patch by deadeyes. M +4 -0 rdphostpreferences.cpp M +5 -0 rdppreferences.ui M +11 -2 rdpview.cpp WebSVN link: http://websvn.kde.org/?view=rev&revision=1330137