Changeset 37860 for main/trunk/greenstone3/web/interfaces/default

Timestamp:

2023-07-31T09:11:24+12:00 (11 months ago)

Author:

anupama

Message:

I'd run a JS change Dr Bainbridge made by Finn, where the spread operator was used in a way not liked by older versions of firefox (running on the local 32 bit linux test VM). I think Finn ok-ed the change, but I seem to have forgotten to commit it after the change was made locally for testing. Committing now.

File:

• main/trunk/greenstone3/web/interfaces/default/js/utility_scripts.js (modified) (1 diff)

main/trunk/greenstone3/web/interfaces/default/js/utility_scripts.js

@@ -1731 +1731 @@
             },
             color: regColour + regionTransparency,
-            ...(selected) && {color: "rgba(255,50,50,0.5)"},
+            //...(selected) && {color: "rgba(255,50,50,0.5)"},
          });
+     // I think I checked with Finn that this fix by Dr Bainbridge is fine:
+     if(selected) {
+         associatedReg.color="rgba(255,50,50,0.5)";
+     }
+      
          data.tempSpeakerObjects[i].region = associatedReg;
          if (selected && data.tempSpeakerObjects[i].locked) { // add padlock to regions if they are selected and locked