Re: [pve-devel] [PATCH pve_flutter_frontend 2/3] feat: ui: add bottom sheet to change VMs/CTs options

["Shan Shaji" <s.shaji@proxmox.com>]

Thanks @Thomas for spotting that. I will send a v2 with the fix. 

On Tue Sep 23, 2025 at 2:03 PM CEST, Thomas Lamprecht wrote:
> Am 23.09.25 um 11:37 schrieb Shan Shaji:
>> On the options page for VMs and CTs it was easy to change the configs by
>> mistake. To avoid add a bottom sheet to show the editable option.
>> 
>> The value will only be updated when the `update` button is pressed.
>> 
>> Signed-off-by: Shan Shaji <s.shaji@proxmox.com>
>> ---
>>  .../pve_config_switch_form.dart               |  54 ++++++
>>  lib/widgets/pve_config_list_tile.dart         | 176 +++++++++++++++++
>>  lib/widgets/pve_lxc_options_widget.dart       |  78 +++++---
>>  lib/widgets/pve_qemu_options_widget.dart      | 177 ++++++++++++------
>>  4 files changed, 400 insertions(+), 85 deletions(-)
>>  create mode 100644 lib/widgets/pve_config_forms/pve_config_switch_form.dart
>>  create mode 100644 lib/widgets/pve_config_list_tile.dart
>> 
>> diff --git a/lib/widgets/pve_config_forms/pve_config_switch_form.dart b/lib/widgets/pve_config_forms/pve_config_switch_form.dart
>> new file mode 100644
>> index 0000000..61ae014
>> --- /dev/null
>> +++ b/lib/widgets/pve_config_forms/pve_config_switch_form.dart
>> @@ -0,0 +1,54 @@
>> +import 'package:flutter/material.dart';
>> +import 'package:pve_flutter_frontend/widgets/pve_config_tile.dart';
>
> Above does not exist, should probably be the newly added "pve_config_list_tile.dart" file?
> I.e.:
>
> diff --git a/lib/widgets/pve_config_forms/pve_config_switch_form.dart b/lib/widgets/pve_config_forms/pve_config_switch_form.dart
> index 61ae014..40bef8c 100644
> --- a/lib/widgets/pve_config_forms/pve_config_switch_form.dart
> +++ b/lib/widgets/pve_config_forms/pve_config_switch_form.dart
> @@ -1,5 +1,5 @@
>  import 'package:flutter/material.dart';
> -import 'package:pve_flutter_frontend/widgets/pve_config_tile.dart';
> +import 'package:pve_flutter_frontend/widgets/pve_config_list_tile.dart';
>  
>  class PveConfigSwitchForm extends StatefulWidget {
>    const PveConfigSwitchForm({



_______________________________________________
pve-devel mailing list
pve-devel@lists.proxmox.com
https://lists.proxmox.com/cgi-bin/mailman/listinfo/pve-devel