Merge pull request #2055 from Gauravjeetsingh/qa-fixes

Bug fixes and improvements

Author: Gauravjeetsingh

www/main/navigator/shabad/FavShabadIcon.jsx

@@ -66,19 +66,22 @@ const FavShabadIcon = ({ paneId }) => {
   }, [favShabad, currentShabad]);
 
   const toggleFavShabad = () => {
+    setLoading(true);
     if (favShabadIndex < 0) {
-      addToFav(currentShabad, currentVerse, userToken);
+      addToFav(currentShabad, currentVerse, userToken)
+        .then(() => fetchFavShabad(userToken))
+        .then((data) => {
+          setFavShabad([...data]);
+          setLoading(false);
+        });
     } else {
-      favShabad.splice(favShabadIndex, 1);
-      removeFromFav(currentShabad, userToken);
-      setFavShabad([...favShabad]);
+      removeFromFav(currentShabad, userToken)
+        .then(() => fetchFavShabad(userToken))
+        .then((data) => {
+          setFavShabad([...data]);
+          setLoading(false);
+        });
     }
-    const fetchProgress = fetchFavShabad(userToken);
-    setLoading(true);
-    fetchProgress.then((data) => {
-      setFavShabad([...data]);
-      setLoading(false);
-    });
   };
 
   if (

www/obs/index.html

@@ -211,8 +211,6 @@
           spanEl.style.backgroundColor = 'transparent';
         });
         document.querySelector('.overlay-wrapper').style.backgroundColor = 'transparent';
-        // Apply the background color from theme
-        el.style.backgroundColor = `rgba(${hexToRgb(overlayPrefs.bgColor)}, ${bgOpacity})`;
       });
 
       // Special style overrides for vertical layout