[pve-devel] [PATCH v6 manager] ui: vm network: allow to override MTU for virtio devices
Thomas Lamprecht
t.lamprecht at proxmox.com
Tue Nov 8 17:47:08 CET 2022
Am 12/10/2022 um 10:57 schrieb Daniel Tschlatscher:
> This patch still applies well and works as intended.
>
> There is one UI suggestion inline.
>
>
> Tested-by: Daniel Tschlatscher <d.tschlatscher at proxmox.com>
>
thx!
>> @@ -161,6 +187,14 @@ Ext.define('PVE.qemu.NetworkInputPanel', {
>> value: '',
>> allowBlank: true,
>> },
>> + {
>> + xtype: 'displayfield',
>> + userCls: 'pmx-hint',
>> + value: gettext("Use the special value '1' to inherit the MTU value from the underlying bridge"),
>> + bind: {
>> + hidden: '{!showMtuHint}',
>> + },
>> + },
> I think it might make a bit more sense to move this into the
> advancedColumn1 as well, as now it is displayed in the opposite column,
> which makes the UI flow kind of confusing.
> Perhaps an even better option though, would be to move the displayfield
> into an advancedColumnB, which would also remove the line width limit
> and with it the somewhat ugly line break.
>
With Oguz not working here anymore, would you like to re-send this with that
change?
More information about the pve-devel
mailing list