Improve handling of the undo/redo shortcuts (#659)

* Prevent Cmd+Z/Cmd+Shift+Z from propagating to the browser

* Allow browser to handle undo/redo shortcuts in input fields

* Don’t change a transaction’s note from null to ''

packages/desktop-client/src/browser-preload.browser.js

@@ -168,6 +168,14 @@ document.addEventListener('keydown', e => {
     }
     // Cmd/Ctrl+z
     else if (e.keyCode === 90) {
+      if (
+        e.target.tagName === 'INPUT' ||
+        e.target.tagName === 'TEXTAREA' ||
+        e.target.isContentEditable
+      ) {
+        return;
+      }
+      e.preventDefault();
       if (e.shiftKey) {
         // Redo
         window.__actionsForMenu.redo();

packages/desktop-client/src/components/accounts/TransactionsTable.js

@@ -557,6 +557,11 @@ export const Transaction = React.memo(function Transaction(props) {
     if (transaction[name] !== value) {
       let newTransaction = { ...transaction, [name]: value };
 
+      // Don't change the note to an empty string if it's null (since they are both rendered the same)
+      if (name === 'note' && value === '' && transaction.note == null) {
+        return;
+      }
+
       if (
         name === 'account' &&
         value &&