fix(popuplayer): Only render PopupReply for staged location (#607)

Author: Conrad Chan

src/popup/PopupLayer.tsx

@@ -89,7 +89,7 @@ const PopupLayer = (props: Props): JSX.Element | null => {
 
     return (
         <>
-            {canCreate && canReply && reference && (
+            {canCreate && canReply && reference && staged && (
                 <div className="ba-PopupLayer-popup">
                     <PopupReply
                         isPending={isPending}

src/popup/__tests__/PopupLayer-test.tsx

@@ -84,10 +84,15 @@ describe('PopupLayer', () => {
             expect(wrapper.exists(PopupReply)).toBe(false);
         });
 
-        test('should render PopupReply if promoting a highlight', () => {
+        test('should render PopupReply if promoting a highlight and staged exists', () => {
             const wrapper = getWrapper({ mode: Mode.NONE, isPromoting: true });
             expect(wrapper.exists(PopupReply)).toBe(true);
         });
+
+        test('should not render PopupReply if promoting a highlight but staged does not exist', () => {
+            const wrapper = getWrapper({ mode: Mode.NONE, isPromoting: true, staged: null });
+            expect(wrapper.exists(PopupReply)).toBe(false);
+        });
     });
 
     describe('event handlers', () => {