all lists on lists.proxmox.com
 help / color / mirror / Atom feed
* [PATCH manager] ui: vCPU flag selector: add tooltips for acceleration modes
@ 2026-07-28  8:52 Fiona Ebner
  2026-07-28 12:05 ` Arthur Bied-Charreton
  2026-07-28 12:18 ` Maximiliano Sandoval
  0 siblings, 2 replies; 3+ messages in thread
From: Fiona Ebner @ 2026-07-28  8:52 UTC (permalink / raw)
  To: pve-devel

Avoid confusion for users who are not familiar with the terms [0].
Especially TCG is not that well-known.

[0]: https://forum.proxmox.com/threads/185360/

Signed-off-by: Fiona Ebner <f.ebner@proxmox.com>
---
 www/manager6/form/VMCPUFlagSelector.js | 17 +++++++++++++++--
 1 file changed, 15 insertions(+), 2 deletions(-)

diff --git a/www/manager6/form/VMCPUFlagSelector.js b/www/manager6/form/VMCPUFlagSelector.js
index aadd8b88..b51731aa 100644
--- a/www/manager6/form/VMCPUFlagSelector.js
+++ b/www/manager6/form/VMCPUFlagSelector.js
@@ -292,8 +292,21 @@ Ext.define('PVE.form.VMCPUFlagSelector', {
                         xtype: 'segmentedbutton',
                         allowMultiple: false,
                         items: [
-                            { text: 'KVM', value: 1, pressed: true },
-                            { text: 'TCG', value: 0 },
+                            {
+                                text: 'KVM',
+                                value: 1,
+                                pressed: true,
+                                tooltip: gettext(
+                                    'Hardware-accelerated virtualization (fast, recommended)',
+                                ),
+                            },
+                            {
+                                text: 'TCG',
+                                value: 0,
+                                tooltip: gettext(
+                                    'Software emulation (slow, needed for cross-architecture emulation)',
+                                ),
+                            },
                         ],
                         listeners: {
                             change: function (field, value) {
-- 
2.47.3





^ permalink raw reply related	[flat|nested] 3+ messages in thread

* Re: [PATCH manager] ui: vCPU flag selector: add tooltips for acceleration modes
  2026-07-28  8:52 [PATCH manager] ui: vCPU flag selector: add tooltips for acceleration modes Fiona Ebner
@ 2026-07-28 12:05 ` Arthur Bied-Charreton
  2026-07-28 12:18 ` Maximiliano Sandoval
  1 sibling, 0 replies; 3+ messages in thread
From: Arthur Bied-Charreton @ 2026-07-28 12:05 UTC (permalink / raw)
  To: Fiona Ebner; +Cc: pve-devel

On Tue, Jul 28, 2026 at 10:52:00AM +0200, Fiona Ebner wrote:
> Avoid confusion for users who are not familiar with the terms [0].
> Especially TCG is not that well-known.
> 
> [0]: https://forum.proxmox.com/threads/185360/
> 
> Signed-off-by: Fiona Ebner <f.ebner@proxmox.com>
> ---
>  www/manager6/form/VMCPUFlagSelector.js | 17 +++++++++++++++--
>  1 file changed, 15 insertions(+), 2 deletions(-)
> 
> diff --git a/www/manager6/form/VMCPUFlagSelector.js b/www/manager6/form/VMCPUFlagSelector.js
> index aadd8b88..b51731aa 100644
> --- a/www/manager6/form/VMCPUFlagSelector.js
> +++ b/www/manager6/form/VMCPUFlagSelector.js
> @@ -292,8 +292,21 @@ Ext.define('PVE.form.VMCPUFlagSelector', {
>                          xtype: 'segmentedbutton',
>                          allowMultiple: false,
>                          items: [
> -                            { text: 'KVM', value: 1, pressed: true },
> -                            { text: 'TCG', value: 0 },
> +                            {
> +                                text: 'KVM',
> +                                value: 1,
> +                                pressed: true,
> +                                tooltip: gettext(
> +                                    'Hardware-accelerated virtualization (fast, recommended)',
> +                                ),
> +                            },
> +                            {
> +                                text: 'TCG',
> +                                value: 0,
> +                                tooltip: gettext(
> +                                    'Software emulation (slow, needed for cross-architecture emulation)',
> +                                ),
> +                            },
>                          ],
>                          listeners: {
>                              change: function (field, value) {
> -- 
> 2.47.3
> 
> 
Thanks for this, good addition and works as advertised. 

Reviewed-by: Arthur Bied-Charreton <a.bied-charreton@proxmox.com>
Tested-by: Arthur Bied-Charreton <a.bied-charreton@proxmox.com>




^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH manager] ui: vCPU flag selector: add tooltips for acceleration modes
  2026-07-28  8:52 [PATCH manager] ui: vCPU flag selector: add tooltips for acceleration modes Fiona Ebner
  2026-07-28 12:05 ` Arthur Bied-Charreton
@ 2026-07-28 12:18 ` Maximiliano Sandoval
  1 sibling, 0 replies; 3+ messages in thread
From: Maximiliano Sandoval @ 2026-07-28 12:18 UTC (permalink / raw)
  To: Fiona Ebner; +Cc: pve-devel

Fiona Ebner <f.ebner@proxmox.com> writes:

> Avoid confusion for users who are not familiar with the terms [0].
> Especially TCG is not that well-known.
>
> [0]: https://forum.proxmox.com/threads/185360/
>
> Signed-off-by: Fiona Ebner <f.ebner@proxmox.com>

I had this on my todo-list. lgtm.

Reviewed-by: Maximiliano Sandoval <m.sandoval@proxmox.com>

-- 
Maximiliano




^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2026-07-28 12:18 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2026-07-28  8:52 [PATCH manager] ui: vCPU flag selector: add tooltips for acceleration modes Fiona Ebner
2026-07-28 12:05 ` Arthur Bied-Charreton
2026-07-28 12:18 ` Maximiliano Sandoval

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.
Service provided by Proxmox Server Solutions GmbH | Privacy | Legal