ui: backup: replication: replace non-clickable checkbox with icons

From a usability view, having a checkbox that is not clickable is pretty
misleading, especially if the visual style is exactly the same as in
other places in the UI where the checkbox is functional.

Signed-off-by: Lukas Wagner <l.wagner@proxmox.com>

diff --git a/www/manager6/dc/Backup.js b/www/manager6/dc/Backup.js
index d0046177ae0c9ddc7f90cb626d9cbbb63a7b8bd6..b210311d36b6dce5e4fc8bab124e2c5a9e5e3281 100644 (file)
--- a/www/manager6/dc/Backup.js
+++ b/www/manager6/dc/Backup.js
@@ -715,11 +715,9 @@ Ext.define('PVE.dc.BackupView', {
                    header: gettext('Enabled'),
                    width: 80,
                    dataIndex: 'enabled',
-                   xtype: 'checkcolumn',
+                   align: 'center',
+                   renderer: Proxmox.Utils.renderEnabledIcon,
                    sortable: true,
-                   disabled: true,
-                   disabledCls: 'x-item-enabled',
-                   stopSelection: false,
                },
                {
                    header: gettext('ID'),

diff --git a/www/manager6/grid/Replication.js b/www/manager6/grid/Replication.js
index b17288b9b6a7c39f2742ce0bbaacecfa4c9e08c4..1e4e00fcbd5bb1d78c625f025f4a651696dd3e14 100644 (file)
--- a/www/manager6/grid/Replication.js
+++ b/www/manager6/grid/Replication.js
@@ -279,11 +279,12 @@ Ext.define('PVE.grid.ReplicaView', {
 
        me.columns = [
            {
-               text: gettext('Enabled'),
+               header: gettext('Enabled'),
+               width: 80,
                dataIndex: 'enabled',
-               xtype: 'checkcolumn',
+               align: 'center',
+               renderer: Proxmox.Utils.renderEnabledIcon,
                sortable: true,
-               disabled: true,
            },
            {
                text: 'ID',