From 70daa2b7e83563f0caed27682519d6fc0986de18 Mon Sep 17 00:00:00 2001 From: Artur Drozdz Date: Mon, 1 Nov 2021 23:38:53 +0100 Subject: [PATCH 1/2] fix: context menu not working when clicking on cell holding number --- src/components/BrowserCell/BrowserCell.react.js | 13 ++++++++++--- 1 file changed, 10 insertions(+), 3 deletions(-) diff --git a/src/components/BrowserCell/BrowserCell.react.js b/src/components/BrowserCell/BrowserCell.react.js index 1961c2ebd0..838b3b78a7 100644 --- a/src/components/BrowserCell/BrowserCell.react.js +++ b/src/components/BrowserCell/BrowserCell.react.js @@ -285,10 +285,17 @@ export default class BrowserCell extends Component { return { text: 'Set filter...', items: constraints.map(constraint => { const definition = Filters.Constraints[constraint]; + const copyableValue = String(this.copyableValue); // Smart ellipsis for value - if it's long trim it in the middle: Lorem ipsum dolor si... aliqua - const value = this.copyableValue.length < 30 ? this.copyableValue : - `${this.copyableValue.substr(0, 20)}...${this.copyableValue.substr(this.copyableValue.length - 7)}`; - const text = `${this.props.field} ${definition.name}${definition.comparable ? (' ' + value) : ''}`; + const value = + copyableValue.length < 30 + ? copyableValue + : `${copyableValue.substr(0, 20)}...${copyableValue.substr( + copyableValue.length - 7 + )}`; + const text = `${this.props.field} ${definition.name}${ + definition.comparable ? ' ' + value : '' + }`; return { text, callback: this.pickFilter.bind(this, constraint) From 5d1bca81ed76025015d5ce8d5960eed1cb258d25 Mon Sep 17 00:00:00 2001 From: Artur Drozdz Date: Mon, 1 Nov 2021 23:48:53 +0100 Subject: [PATCH 2/2] Fixing React key warning --- .../ContextMenu/ContextMenu.react.js | 34 ++++++++++++++----- 1 file changed, 25 insertions(+), 9 deletions(-) diff --git a/src/components/ContextMenu/ContextMenu.react.js b/src/components/ContextMenu/ContextMenu.react.js index d8850500d8..64687510b5 100644 --- a/src/components/ContextMenu/ContextMenu.react.js +++ b/src/components/ContextMenu/ContextMenu.react.js @@ -51,16 +51,32 @@ const MenuSection = ({ level, items, path, setPath, hide }) => { return (); }