Allow deselection of selected items

Signed-off-by: Axel Boberg <[email protected]>

api/browser/client.js

@@ -181,9 +181,9 @@ function subtractSelection (item) {
  * @param { String } item The id of an item to check
  * @returns { Boolean }
  */
-function isSelected (item) {
+async function isSelected (item) {
   assertIdentity()
-  const selection = state.getLocalState()?._connections?.[getIdentity()]?.selection
+  const selection = await state.get(`_connections.${getIdentity()}.selection`)
   if (!selection) {
     return false
   }

docs/api/README.md

@@ -364,7 +364,7 @@ Add one or more items to the selecton by their ids.
 **Only available within the render process**  
 Subtract one or more items to the selecton by their ids.
 
-### `bridge.client.isSelected(itemId): Boolean`  
+### `bridge.client.isSelected(itemId): Promise<Boolean>`  
 **Only available within the render process**  
 Check whether or not an item is selected by the current client.  
 

plugins/rundown/app/components/RundownList/index.jsx

@@ -248,12 +248,24 @@ export function RundownList ({
     bridge.commands.executeCommand('rundown.moveItem', rundownId, newIndex, itemId)
   }
 
-  function handleFocus (itemId) {
+  async function handleFocus (itemId) {
     if (keyboard.keyIsPressed('meta')) {
-      bridge.client.addSelection(itemId)
-    } else {
-      bridge.client.setSelection(itemId)
+      const isSelected = await bridge.client.isSelected(itemId)
+      if (isSelected) {
+        bridge.client.subtractSelection(itemId)
+      } else {
+        bridge.client.addSelection(itemId)
+      }
+      return
+    }
+
+    if (keyboard.keyIsPressed('shift')) {
+      // Select all items between the last selection and the new item
+      // Check data-item-id
+      return
     }
+
+    bridge.client.setSelection(itemId)
   }
 
   function handleFocusPropagation (e) {