[pve-devel] [PATCH pve_flutter_frontend v2] fix: ui: render pending chip widget in column to avoid overflow error

[pve-devel] [PATCH pve_flutter_frontend v2] fix: ui: render pending chip widget in column to avoid overflow error

[Shan Shaji]

When the pending chip appears after enabling an option from the options
page, Flutter throws an overflow error, causing the chip to render
on top of the Switch widget due to insufficient space between
the title and the switch toggle.

To resolve this, modify the layout so that the pending chip widget
is rendered in a column instead of a row.

Signed-off-by: Shan Shaji <s.shaji@proxmox.com>
---
 
 Steps to reproduce the issue:
 1. Navigate to the options tab of any VM.
 2. Enable the "KVM hardware viritualization" option.
 3. You will encounter a render flex error.
 
 changes since v1:
 * changed commit message. 
 * moved steps to reproduce section under "---".
 
 lib/widgets/pve_config_switch_list_tile.dart | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/lib/widgets/pve_config_switch_list_tile.dart b/lib/widgets/pve_config_switch_list_tile.dart
index af02ee6..c209fbe 100644
--- a/lib/widgets/pve_config_switch_list_tile.dart
+++ b/lib/widgets/pve_config_switch_list_tile.dart
@@ -32,8 +32,8 @@ class PveConfigSwitchListTile extends StatelessWidget {
 
   Widget? _getTitle() {
     if (pending != null) {
-      return Row(
-        mainAxisAlignment: MainAxisAlignment.spaceBetween,
+      return Column(
+        crossAxisAlignment: CrossAxisAlignment.start,
         children: [
           title!,
           Chip(
-- 
2.39.5 (Apple Git-154)



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

[pve-devel] applied: [PATCH pve_flutter_frontend v2] fix: ui: render pending chip widget in column to avoid overflow error

[Dominik Csapak]

On 6/16/25 14:58, Shan Shaji wrote:
> When the pending chip appears after enabling an option from the options
> page, Flutter throws an overflow error, causing the chip to render
> on top of the Switch widget due to insufficient space between
> the title and the switch toggle.
> 
> To resolve this, modify the layout so that the pending chip widget
> is rendered in a column instead of a row.
> 
> Signed-off-by: Shan Shaji <s.shaji@proxmox.com>
> ---
>   
>   Steps to reproduce the issue:
>   1. Navigate to the options tab of any VM.
>   2. Enable the "KVM hardware viritualization" option.
>   3. You will encounter a render flex error.
>   
>   changes since v1:
>   * changed commit message.
>   * moved steps to reproduce section under "---".
>   
>   lib/widgets/pve_config_switch_list_tile.dart | 4 ++--
>   1 file changed, 2 insertions(+), 2 deletions(-)
> 
>

applied, thanks!


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