Re: [pve-devel] novnc update

2015-08-31 Thread Alexandre DERUMIER
>>I just tested it, does not really work (shows garbage). see screenshot 
>>attached. tested on IE11, FF, Chrome - all on win7.

 Don't have problem here (archlinux - firefox/chrome). Is it windows specific ?




- Mail original -
De: "Martin Maurer" <mar...@proxmox.com>
À: "pve-devel" <pve-devel@pve.proxmox.com>
Envoyé: Vendredi 28 Août 2015 12:29:44
Objet: Re: [pve-devel] novnc update

I just tested it, does not really work (shows garbage). see screenshot 
attached. tested on IE11, FF, Chrome - all on win7. 

the embedded console on IE11 does always reload. 

So due to the fact that the console already worked great before, I 
suggest to go back and revert this for now. 

Martin 


On 25.08.2015 06:59, Alexandre Derumier wrote: 
> Theses patches update novnc to last git version. 
> 
> They are some performance improvements 
> 
> and also a new fullscreen button has been added. 
> 
> 
> Currently in proxmox, the fullscreen button don't work in embedded console. 
> For the popup console, it should work, but because of current proxmox resize 
> windows code, 
> it's not working 
> 
> If the console is called with resize=scale, the fullscreen button is working: 
> https://hostname:8006/?console=kvm=1=100=null=node1=scale
>  
> 
> 
> ___ 
> pve-devel mailing list 
> pve-devel@pve.proxmox.com 
> http://pve.proxmox.com/cgi-bin/mailman/listinfo/pve-devel 
> 

___ 
pve-devel mailing list 
pve-devel@pve.proxmox.com 
http://pve.proxmox.com/cgi-bin/mailman/listinfo/pve-devel 
___
pve-devel mailing list
pve-devel@pve.proxmox.com
http://pve.proxmox.com/cgi-bin/mailman/listinfo/pve-devel


Re: [pve-devel] novnc update

2015-08-31 Thread Alexandre DERUMIER
ok, I can reproduce it now.

I'll try to bissect this
- Mail original -
De: "Martin Maurer" <mar...@proxmox.com>
À: "aderumier" <aderum...@odiso.com>
Cc: "pve-devel" <pve-devel@pve.proxmox.com>
Envoyé: Lundi 31 Août 2015 12:22:33
Objet: Re: [pve-devel] novnc update

On 31.08.2015 12:12, Alexandre DERUMIER wrote: 
>>> I just tested it, does not really work (shows garbage). see screenshot 
>>> attached. tested on IE11, FF, Chrome - all on win7. 
> 
> Don't have problem here (archlinux - firefox/chrome). Is it windows specific 
> ? 

No, we also see this on latest FF on Debian Jessie (we used FF from MINT 
project) or any other Linux OS. 

You need to: 
Open the embedded console of a VM, click on fullscreen and go back - 
garbage starts. 

-- 
Best Regards, 

Martin Maurer 

___
pve-devel mailing list
pve-devel@pve.proxmox.com
http://pve.proxmox.com/cgi-bin/mailman/listinfo/pve-devel


Re: [pve-devel] novnc update

2015-08-31 Thread Alexandre DERUMIER
It's working fine for me at this commit (fullscreen support)
https://github.com/kanaka/noVNC/commit/340290fa1d6a53d30a38b6604bcacef5940c4c52


but garbage occur since this optimisation commit
https://github.com/kanaka/noVNC/commit/abf2b09ea7ca59d006c250ea96c5f219f83ca7d0


I'll make a patch to revert to the fullscreen commit.


- Mail original -
De: "aderumier" <aderum...@odiso.com>
À: "Martin Maurer" <mar...@proxmox.com>
Cc: "pve-devel" <pve-devel@pve.proxmox.com>
Envoyé: Lundi 31 Août 2015 12:55:20
Objet: Re: [pve-devel] novnc update

ok, I can reproduce it now. 

I'll try to bissect this 
- Mail original - 
De: "Martin Maurer" <mar...@proxmox.com> 
À: "aderumier" <aderum...@odiso.com> 
Cc: "pve-devel" <pve-devel@pve.proxmox.com> 
Envoyé: Lundi 31 Août 2015 12:22:33 
Objet: Re: [pve-devel] novnc update 

On 31.08.2015 12:12, Alexandre DERUMIER wrote: 
>>> I just tested it, does not really work (shows garbage). see screenshot 
>>> attached. tested on IE11, FF, Chrome - all on win7. 
> 
> Don't have problem here (archlinux - firefox/chrome). Is it windows specific 
> ? 

No, we also see this on latest FF on Debian Jessie (we used FF from MINT 
project) or any other Linux OS. 

You need to: 
Open the embedded console of a VM, click on fullscreen and go back - 
garbage starts. 

-- 
Best Regards, 

Martin Maurer 

___ 
pve-devel mailing list 
pve-devel@pve.proxmox.com 
http://pve.proxmox.com/cgi-bin/mailman/listinfo/pve-devel 
___
pve-devel mailing list
pve-devel@pve.proxmox.com
http://pve.proxmox.com/cgi-bin/mailman/listinfo/pve-devel


Re: [pve-devel] novnc update

2015-08-31 Thread Martin Maurer

On 31.08.2015 12:12, Alexandre DERUMIER wrote:

I just tested it, does not really work (shows garbage). see screenshot
attached. tested on IE11, FF, Chrome - all on win7.


  Don't have problem here (archlinux - firefox/chrome). Is it windows specific ?


No, we also see this on latest FF on Debian Jessie (we used FF from MINT 
project) or any other Linux OS.


You need to:
Open the embedded console of a VM, click on fullscreen and go back - 
garbage starts.


--
Best Regards,

Martin Maurer

___
pve-devel mailing list
pve-devel@pve.proxmox.com
http://pve.proxmox.com/cgi-bin/mailman/listinfo/pve-devel


Re: [pve-devel] novnc update

2015-08-28 Thread Dietmar Maurer
 But It's really usefull for embedded console.

OK, applied. Thanks!

___
pve-devel mailing list
pve-devel@pve.proxmox.com
http://pve.proxmox.com/cgi-bin/mailman/listinfo/pve-devel


Re: [pve-devel] novnc update

2015-08-26 Thread Dietmar Maurer

Why do we add the button if it does not work (confused)?

On 08/25/2015 06:59 AM, Alexandre Derumier wrote:

Currently in proxmox, the fullscreen button don't work in embedded console.
For the popup console, it should work, but because of current proxmox resize 
windows code,
it's not working





___
pve-devel mailing list
pve-devel@pve.proxmox.com
http://pve.proxmox.com/cgi-bin/mailman/listinfo/pve-devel


Re: [pve-devel] novnc update

2015-08-26 Thread Alexandre DERUMIER
Why do we add the button if it does not work (confused)? 

I've got it working in embedded console now. (see my other patch for 
pve-manager).

For popup console, i'm still working on it.
(It's going fullscreen, but proxmox windows resize code shrink them just after.
I'm trying to disable proxmox windows resize when i click on full screen, but 
don't seem to work.)
If I can get it working in popup, I'll send a patch to remove the button in 
this case.
But It's really usefull for embedded console.



- Mail original -
De: dietmar diet...@proxmox.com
À: aderumier aderum...@odiso.com, pve-devel pve-devel@pve.proxmox.com
Envoyé: Mercredi 26 Août 2015 16:34:15
Objet: Re: [pve-devel] novnc update

Why do we add the button if it does not work (confused)? 

On 08/25/2015 06:59 AM, Alexandre Derumier wrote: 
 Currently in proxmox, the fullscreen button don't work in embedded console. 
 For the popup console, it should work, but because of current proxmox resize 
 windows code, 
 it's not working 
 
 

___
pve-devel mailing list
pve-devel@pve.proxmox.com
http://pve.proxmox.com/cgi-bin/mailman/listinfo/pve-devel


[pve-devel] novnc update

2015-08-24 Thread Alexandre Derumier
Theses patches update novnc to last git version.

They are some performance improvements

and also a new fullscreen button has been added.


Currently in proxmox, the fullscreen button don't work in embedded console.
For the popup console, it should work, but because of current proxmox resize 
windows code,
it's not working

If the console is called with resize=scale, the fullscreen button is working:
https://hostname:8006/?console=kvmnovnc=1vmid=100vmname=nullnode=node1resize=scale

___
pve-devel mailing list
pve-devel@pve.proxmox.com
http://pve.proxmox.com/cgi-bin/mailman/listinfo/pve-devel


[pve-devel] novnc update

2015-08-24 Thread Alexandre Derumier
Theses patches update novnc to last git version.

They are some performance improvements

and also a new fullscreen button has been added.


Currently in proxmox, the fullscreen button don't work in embedded console.
For the popup console, it should work, but because of current proxmox resize 
windows code,
it's not working

If the console is called with resize=scale, the fullscreen button is working:
https://hostname:8006/?console=kvmnovnc=1vmid=100vmname=nullnode=node1resize=scale


___
pve-devel mailing list
pve-devel@pve.proxmox.com
http://pve.proxmox.com/cgi-bin/mailman/listinfo/pve-devel


Re: [pve-devel] novnc update ?

2015-06-23 Thread Dietmar Maurer

Yes, that would be great - any volunteer?

On 06/23/2015 11:53 AM, Alexandre DERUMIER wrote:

Hi,

I wonder if it could be great to upgrade novnc from 0.4 to 0.5.1 ?

They are a lot of dix for keyboards, tightvnc protocol support




___
pve-devel mailing list
pve-devel@pve.proxmox.com
http://pve.proxmox.com/cgi-bin/mailman/listinfo/pve-devel


Re: [pve-devel] novnc update ?

2015-06-23 Thread Alexandre DERUMIER
I have begin to look at code, but don't work yet.
(I can connect but display black).

They are also new display resizing features.
- Mail original -
De: dietmar diet...@proxmox.com
À: aderumier aderum...@odiso.com, pve-devel pve-devel@pve.proxmox.com
Envoyé: Mardi 23 Juin 2015 17:19:07
Objet: Re: [pve-devel] novnc update ?

Yes, that would be great - any volunteer? 

On 06/23/2015 11:53 AM, Alexandre DERUMIER wrote: 
 Hi, 
 
 I wonder if it could be great to upgrade novnc from 0.4 to 0.5.1 ? 
 
 They are a lot of dix for keyboards, tightvnc protocol support 
 

___
pve-devel mailing list
pve-devel@pve.proxmox.com
http://pve.proxmox.com/cgi-bin/mailman/listinfo/pve-devel


[pve-devel] novnc update ?

2015-06-23 Thread Alexandre DERUMIER
Hi,

I wonder if it could be great to upgrade novnc from 0.4 to 0.5.1 ?

They are a lot of dix for keyboards, tightvnc protocol support



https://github.com/kanaka/noVNC/releases

Major Changes Since 0.5

Improved keyboard support via use of keysyms.js
Fix HEXTILE blank tile support
lower minimum width to accommodate smaller screens
Fix issue causing true-color cursors to not be changed
Fix subprotocol support for the TIGHT encoding

Major Changes Since v0.4

General Fixes/Improvements

Use Object.defineProperty on Array prototype to prevent issues with enumeration
Internal Refactor for improved testability -- NOTE: code which interfaces 
directly with noVNC may see minor breakage (e.g. custom UI elements, etc). 
Please report any bugs you find
Improved parsing of query string parameters
Support container page scrolling
Fixed IE10 keyboard support
Fixes for mobile keyboards (Android and iOS)
Improved foreign keyboard support
dynamic script loading without document.write
Protocol Support

TightVNC support (we support choosing the NOTUNNEL type if tunnels are 
requested)
XVP Support
___
pve-devel mailing list
pve-devel@pve.proxmox.com
http://pve.proxmox.com/cgi-bin/mailman/listinfo/pve-devel


Re: [pve-devel] novnc update ?

2015-06-23 Thread Wolfgang Bumiller
On Tue, Jun 23, 2015 at 11:53:41AM +0200, Alexandre DERUMIER wrote:
 Hi,
 
 I wonder if it could be great to upgrade novnc from 0.4 to 0.5.1 ?

Seeing how I've been having some unpredictable keyboard failures with
noVNC I'd like to see if this makes a difference.

 They are a lot of dix for keyboards, tightvnc protocol support
 
 
 
 https://github.com/kanaka/noVNC/releases
 
 Major Changes Since 0.5
 
 Improved keyboard support via use of keysyms.js
 Fix HEXTILE blank tile support
 lower minimum width to accommodate smaller screens
 Fix issue causing true-color cursors to not be changed
 Fix subprotocol support for the TIGHT encoding
 
 Major Changes Since v0.4
 
 General Fixes/Improvements
 
 Use Object.defineProperty on Array prototype to prevent issues with 
 enumeration
 Internal Refactor for improved testability -- NOTE: code which interfaces 
 directly with noVNC may see minor breakage (e.g. custom UI elements, etc). 
 Please report any bugs you find
 Improved parsing of query string parameters
 Support container page scrolling
 Fixed IE10 keyboard support
 Fixes for mobile keyboards (Android and iOS)
 Improved foreign keyboard support
 dynamic script loading without document.write
 Protocol Support
 
 TightVNC support (we support choosing the NOTUNNEL type if tunnels are 
 requested)
 XVP Support
 ___
 pve-devel mailing list
 pve-devel@pve.proxmox.com
 http://pve.proxmox.com/cgi-bin/mailman/listinfo/pve-devel
 

___
pve-devel mailing list
pve-devel@pve.proxmox.com
http://pve.proxmox.com/cgi-bin/mailman/listinfo/pve-devel


Re: [pve-devel] novnc update ?

2015-06-23 Thread Alexandre DERUMIER
I have send a patch for 0.5.1. (very small change)


master branch is a little bit more complex to port (but have the cool resize 
feature).

But I can't get display working, not sure if it's a bug or not.


- Mail original -
De: aderumier aderum...@odiso.com
À: dietmar diet...@proxmox.com
Cc: pve-devel pve-devel@pve.proxmox.com
Envoyé: Mardi 23 Juin 2015 22:48:15
Objet: Re: [pve-devel] novnc update ?

I have begin to look at code, but don't work yet. 
(I can connect but display black). 

They are also new display resizing features. 
- Mail original - 
De: dietmar diet...@proxmox.com 
À: aderumier aderum...@odiso.com, pve-devel pve-devel@pve.proxmox.com 
Envoyé: Mardi 23 Juin 2015 17:19:07 
Objet: Re: [pve-devel] novnc update ? 

Yes, that would be great - any volunteer? 

On 06/23/2015 11:53 AM, Alexandre DERUMIER wrote: 
 Hi, 
 
 I wonder if it could be great to upgrade novnc from 0.4 to 0.5.1 ? 
 
 They are a lot of dix for keyboards, tightvnc protocol support 
 

___ 
pve-devel mailing list 
pve-devel@pve.proxmox.com 
http://pve.proxmox.com/cgi-bin/mailman/listinfo/pve-devel 

___
pve-devel mailing list
pve-devel@pve.proxmox.com
http://pve.proxmox.com/cgi-bin/mailman/listinfo/pve-devel