fix(hyprland-visual-editor): ajuste de última hora en la lógica de archivos

Author: XimoCP

hyprland-visual-editor/BarWidget.qml

@@ -7,20 +7,25 @@ import qs.Widgets
 NIconButton {
   id: root
 
+  // ── Injected Properties ──────────────────────────────────────────────
   property var pluginApi: null
   property ShellScreen screen
   property string widgetId: ""
   property string section: ""
   property int sectionWidgetIndex: -1
   property int sectionWidgetsCount: 0
 
+  // ── Configuration Logic (Fallback Pattern) ──────────────────────────
   property var cfg: pluginApi?.pluginSettings || ({})
   property var defaults: pluginApi?.manifest?.metadata?.defaultSettings || ({})
 
   readonly property string iconKey: cfg.icon ?? defaults.icon ?? "adjustments-horizontal"
   readonly property string iconColorKey: cfg.iconColor ?? defaults.iconColor ?? "primary"
 
+  // ── Visual Configuration ─────────────────────────────────────────────────
   icon: iconKey
+  
+  // NIconButton already manages tooltips automatically with these two properties:
   tooltipText: pluginApi?.tr("widget.tooltip")
   tooltipDirection: BarService.getTooltipDirection(screen?.name)
 
@@ -29,6 +34,7 @@ NIconButton {
 
   colorBg: Style.capsuleColor
 
+  // Color resolution with transparency protection (Alpha check)
   colorFg: {
     let resolved = Color.resolveColorKeyOptional(iconColorKey);
     if (root.containsMouse) return Color.mOnHover;
@@ -38,27 +44,22 @@ NIconButton {
   border.color: Style.capsuleBorderColor
   border.width: Style.borderS
 
+  // Color transition smoothing
   Behavior on colorFg {
     ColorAnimation { 
       duration: Style.animationFast
       easing.type: Easing.InOutQuad 
     }
   }
 
+  // ── Interacciones ────────────────────────────────────────────────────────
   onClicked: {
     if (pluginApi) {
       pluginApi.openPanel(root.screen, this);
     }
   }
 
-  onEntered: {
-    TooltipService.show(root, pluginApi?.tr("widget.tooltip"), BarService.getTooltipDirection(screen?.name))
-  }
-  
-  onExited: {
-    TooltipService.hide()
-  }
-
+  // Context menu (Right click)
   NPopupContextMenu {
     id: contextMenu
 

hyprland-visual-editor/Settings.qml

@@ -8,15 +8,19 @@ ColumnLayout {
   id: root
   property var pluginApi: null
 
-  // 1. Local state ('edit' convention and official fallbacks)
-  property string editOverlayPath: pluginApi?.pluginSettings?.overlayPath || pluginApi?.manifest?.metadata?.defaultSettings?.overlayPath || "~/.cache/noctalia/HVE/overlay.conf"
-  property bool editAutoApply: pluginApi?.pluginSettings?.autoApply ?? pluginApi?.manifest?.metadata?.defaultSettings?.autoApply ?? true
-  property string editIcon: pluginApi?.pluginSettings?.icon || pluginApi?.manifest?.metadata?.defaultSettings?.icon || "adjustments-horizontal"
-  property string editIconColor: pluginApi?.pluginSettings?.iconColor || pluginApi?.manifest?.metadata?.defaultSettings?.iconColor || "primary"
+  // Settings and default values (Official Noctalia pattern)
+  readonly property var cfg: pluginApi?.pluginSettings || ({})
+  readonly property var defaults: pluginApi?.manifest?.metadata?.defaultSettings || ({})
+
+  // 1. Local state ('edit' convention to avoid unnecessary disk writes)
+  property string editOverlayPath: cfg.overlayPath ?? defaults.overlayPath ?? "~/.cache/noctalia/HVE/overlay.conf"
+  property bool editAutoApply: cfg.autoApply ?? defaults.autoApply ?? true
+  property string editIcon: cfg.icon ?? defaults.icon ?? "adjustments-horizontal"
+  property string editIconColor: cfg.iconColor ?? defaults.iconColor ?? "primary"
 
   spacing: Style.marginM
 
-  // ── Preview ────────────────────────────────────────────────────────────────
+  // ── Preview ──────────────────────────────────────────────────────────
   RowLayout {
     spacing: Style.marginM
     Layout.alignment: Qt.AlignHCenter
@@ -38,7 +42,7 @@ ColumnLayout {
     }
   }
 
-  // ── Icon Configuration ───────────────────────────────────────────────────
+  // ── Icon Configuration ────────────────────────────────────────────────
   NButton {
     Layout.fillWidth: true
     text: pluginApi?.tr("settings.change_icon_button")
@@ -58,12 +62,12 @@ ColumnLayout {
     label: pluginApi?.tr("settings.icon_color_label")
     currentKey: root.editIconColor
     onSelected: key => { root.editIconColor = key }
-    defaultValue: pluginApi?.manifest?.metadata?.defaultSettings?.iconColor || "primary"
+    defaultValue: defaults.iconColor || "primary"
   }
 
   NDivider { Layout.fillWidth: true }
 
-  // ── Files and Application Configuration (Recovered Section) ────────────────
+  // ── Files and Application Configuration ────────────────────────────────
   NTextInput {
     Layout.fillWidth: true
     label: pluginApi?.tr("settings.path_label")
@@ -81,7 +85,7 @@ ColumnLayout {
     onToggled: checked => { root.editAutoApply = checked }
   }
 
-  // ── Save Function ────────────────────────────────────────────────────────
+  // ── Save Function (Required by the Shell) ──────────────────────────
   function saveSettings() {
     if (!pluginApi) return