@sirendesign/markup 1.0.2 → 1.0.4

package/dist/index.js

@@ -37515,35 +37515,39 @@ const useFirebaseSync = () => {
 
 const MarkupWidget = ({ config: userConfig, }) => {
     var _a;
+    // Initialize Firebase BEFORE any hooks run (using useMemo ensures this happens first)
+    useMemo(() => {
+        if ((userConfig === null || userConfig === void 0 ? void 0 : userConfig.firebaseConfig) && getApps().length === 0) {
+            try {
+                initializeFirebase(userConfig.firebaseConfig);
+                console.log('Firebase initialized from MarkupWidget config');
+            }
+            catch (error) {
+                console.error('Failed to initialize Firebase:', error);
+            }
+        }
+        return null;
+    }, [userConfig === null || userConfig === void 0 ? void 0 : userConfig.firebaseConfig]);
     const { isOpen, setIsOpen, activeTab, setActiveTab, selectedFeedback, setSelectedFeedback, feedbackItems, addFeedbackItem, config, setConfig, reset, isAuthenticated, currentUser, } = useMarkupStore();
     // Initialize Firebase sync
     useFirebaseSync();
     require$$0.useEffect(() => {
         if (userConfig) {
             setConfig(userConfig);
-            // Auto-initialize Firebase if config provided and not already initialized
-            if (userConfig.firebaseConfig && getApps().length === 0) {
-                try {
-                    initializeFirebase(userConfig.firebaseConfig);
-                }
-                catch (error) {
-                    console.error('Failed to initialize Firebase:', error);
-                }
-            }
         }
     }, [userConfig, setConfig]);
     require$$0.useEffect(() => {
         const handleKeyDown = (e) => {
             const shortcut = config.shortcut || "ctrl+shift+m";
-            const parts = shortcut.toLowerCase().split("+");
-            const ctrlRequired = parts.includes("ctrl") || parts.includes("control");
-            const shiftRequired = parts.includes("shift");
-            const altRequired = parts.includes("alt");
-            const key = parts[parts.length - 1];
-            if ((ctrlRequired ? e.ctrlKey || e.metaKey : true) &&
-                (shiftRequired ? e.shiftKey : true) &&
-                (altRequired ? e.altKey : true) &&
-                e.key.toLowerCase() === key) {
+            const keys = shortcut.split("+").map((k) => k.trim().toLowerCase());
+            const ctrl = keys.includes("ctrl");
+            const shift = keys.includes("shift");
+            const alt = keys.includes("alt");
+            const key = keys[keys.length - 1];
+            if (e.key.toLowerCase() === key &&
+                e.ctrlKey === ctrl &&
+                e.shiftKey === shift &&
+                e.altKey === alt) {
                 e.preventDefault();
                 setIsOpen(!isOpen);
             }