1540094 - [remote-dbg-next] SidebarRuntimeItem could take a runtime as prop

Status: NEW

(DevTools :: about:debugging, enhancement, P3)

Description

[Julian Descottes [:jdescottes]]

The SidebarRuntimeItem can only render runtimes.

Right now we are not passing a runtime directly as a prop and the parent component has to implement the logic to translate a runtime object to the props expected by SidebarRuntimeItem:

      const keyId = `${runtime.type}-${runtime.id}`;
      const runtimeHasDetails = !!runtime.runtimeDetails;
      const isSelected = selectedPage === PAGE_TYPES.RUNTIME &&
        runtime.id === selectedRuntimeId;

      let name = runtime.name;
      if (runtime.type === RUNTIMES.USB && runtimeHasDetails) {
        // Update the name to be same to the runtime page.
        name = runtime.runtimeDetails.info.name;
      }

      return SidebarRuntimeItem({
        deviceName: runtime.extra.deviceName,
        dispatch,
        icon,
        key: keyId,
        isConnected: runtimeHasDetails,
        isSelected,
        isUnknown: runtime.isUnknown,
        name,
        runtimeId: runtime.id,
      });

link

All this logic could be done in the specialized SidebarRuntimeItem component and we could simply call it as:

      const keyId = `${runtime.type}-${runtime.id}`;
      return SidebarRuntimeItem({
        dispatch,
        key: keyId,
        isSelected,
        runtime,
      });

Comment 1

[Julian Descottes [:jdescottes]]

Will do some cleanup around runtimes for Bug 1542286, otherwise it gets really hard to follow.