Skip to content

Commit e4b55cb

Browse files
authored
Fix matching named monitors opening 2 context menus (#1037)
1 parent ff6bc4f commit e4b55cb

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

src/components/monitor/monitor.jsx

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -45,7 +45,7 @@ const MonitorComponent = props => (
4545
// TW: if export is defined, we always show it, even outside of the editor
4646
disable={!props.draggable && !props.onExport}
4747
holdToDisplay={props.mode === 'slider' ? -1 : 1000}
48-
id={`monitor-${props.label}`}
48+
id={`monitor-${props.id}`}
4949
>
5050
<Draggable
5151
bounds=".monitor-overlay" // Class for monitor container
@@ -75,7 +75,7 @@ const MonitorComponent = props => (
7575
// positioning conflicts between the monitors `transform: scale` and
7676
// the context menus `position: fixed`. For more details, see
7777
// http://meyerweb.com/eric/thoughts/2011/09/12/un-fixing-fixed-elements-with-css-transforms/
78-
<ContextMenu id={`monitor-${props.label}`}>
78+
<ContextMenu id={`monitor-${props.id}`}>
7979
{props.draggable && props.onSetModeToDefault &&
8080
<MenuItem onClick={props.onSetModeToDefault}>
8181
<FormattedMessage

0 commit comments

Comments
 (0)