Hyprpanel now accurately identifies the submap on startup. (#317)

* Hyprpanel now accurately identifies the submap on startup.

* Update default submap logic.

* Separate logic from component

customModules/submap/helpers.ts

@@ -0,0 +1,20 @@
+import { Variable } from 'types/variable';
+
+const hyprland = await Service.import('hyprland');
+
+export const isSubmapEnabled = (submap: string, enabled: string, disabled: string): string => {
+    return submap !== 'default' ? enabled : disabled;
+};
+
+export const getInitialSubmap = (submapStatus: Variable<string>): void => {
+    let submap = hyprland.message('submap');
+
+    const newLineCarriage = /\n/g;
+    submap = submap.replace(newLineCarriage, '');
+
+    if (submap === 'unknown request') {
+        submap = 'default';
+    }
+
+    submapStatus.value = submap;
+};

customModules/submap/index.ts

@@ -7,7 +7,8 @@ import Button from 'types/widgets/button';
 import { Variable as VariableType } from 'types/variable';
 import { Attribute, Child } from 'lib/types/widget';
 import { BarBoxChild } from 'lib/types/bar';
-import { caapitalizeFirstLetter } from 'lib/utils';
+import { capitalizeFirstLetter } from 'lib/utils';
+import { getInitialSubmap, isSubmapEnabled } from './helpers';
 
 const {
     label,
@@ -23,18 +24,24 @@ const {
     scrollDown,
 } = options.bar.customModules.submap;
 
-const submapStatus: VariableType<string> = Variable('');
+const submapStatus: VariableType<string> = Variable('default');
 
 hyprland.connect('submap', (_, currentSubmap) => {
-    submapStatus.value = currentSubmap;
+    if (currentSubmap.length === 0) {
+        submapStatus.value = 'default';
+    } else {
+        submapStatus.value = currentSubmap;
+    }
 });
 
+getInitialSubmap(submapStatus);
+
 export const Submap = (): BarBoxChild => {
     const submapModule = module({
         textIcon: Utils.merge(
             [submapStatus.bind('value'), enabledIcon.bind('value'), disabledIcon.bind('value')],
             (status, enabled, disabled) => {
-                return status.length > 0 ? enabled : disabled;
+                return isSubmapEnabled(status, enabled, disabled);
             },
         ),
         tooltipText: Utils.merge(
@@ -46,9 +53,9 @@ export const Submap = (): BarBoxChild => {
             ],
             (status, enabled, disabled, showSmName) => {
                 if (showSmName) {
-                    return status.length > 0 ? caapitalizeFirstLetter(status) : 'Default';
+                    return capitalizeFirstLetter(status);
                 }
-                return status.length > 0 ? enabled : disabled;
+                return isSubmapEnabled(status, enabled, disabled);
             },
         ),
         boxClass: 'submap',
@@ -61,9 +68,9 @@ export const Submap = (): BarBoxChild => {
             ],
             (status, enabled, disabled, showSmName) => {
                 if (showSmName) {
-                    return status.length > 0 ? caapitalizeFirstLetter(status) : 'Default';
+                    return capitalizeFirstLetter(status);
                 }
-                return status.length > 0 ? enabled : disabled;
+                return isSubmapEnabled(status, enabled, disabled);
             },
         ),
         showLabelBinding: label.bind('value'),

lib/utils.ts

@@ -190,6 +190,6 @@ export const isValidGjsColor = (color: string): boolean => {
     return false;
 };
 
-export const caapitalizeFirstLetter = (str: string): string => {
+export const capitalizeFirstLetter = (str: string): string => {
     return str.charAt(0).toUpperCase() + str.slice(1);
 };