note and slight workaround for a bug with chat; also don't show upload previes for markdown, since in some cases they are really annoying

Author: williamstein

src/packages/frontend/chat/input.tsx

@@ -156,10 +156,17 @@ export default function ChatInput({
       enableMentions={true}
       submitMentionsRef={submitMentionsRef}
       onChange={(input) => {
+        /* BUG: in Markdown mode this stops getting
+        called after you paste in an image.  It works
+        fine in Slate/Text mode. See
+        https://github.com/sagemathinc/cocalc/issues/7728
+        */
         setInput(input);
         saveChat(input);
       }}
       onShiftEnter={(input) => {
+        setInput(input);
+        saveChat(input);
         saveChat.cancel();
         on_send(input);
       }}

src/packages/frontend/editors/slate/upload.tsx

@@ -98,12 +98,12 @@ export default function useUpload(
 
   return (
     <BlobUpload
+      show_upload={false}
       className="smc-vfill"
       project_id={project_id}
       event_handlers={updloadEventHandlers}
       style={{ height: "100%", width: "100%" }}
       dropzone_ref={dropzoneRef}
-      show_upload={true}
     >
       {body}
     </BlobUpload>

src/packages/jupyter/redux/actions.ts

@@ -379,6 +379,7 @@ export abstract class JupyterActions extends Actions<JupyterStoreState> {
   // Set the input of the given cell in the syncdb, which will also change the store.
   // Might throw a CellWriteProtectedException
   public set_cell_input(id: string, input: string, save = true): void {
+    if (!this.store) return;
     if (this.store.getIn(["cells", id, "input"]) == input) {
       // nothing changed.   Note, I tested doing the above check using
       // both this.syncdb and this.store, and this.store is orders of magnitude faster.