diff --git a/src/button/ReactionToggleButton.module.css b/src/button/ReactionToggleButton.module.css index 384d8906..a4b978bf 100644 --- a/src/button/ReactionToggleButton.module.css +++ b/src/button/ReactionToggleButton.module.css @@ -41,9 +41,6 @@ div.reactionPopupMenuRoot.reactionPopupMenuModal > div > div { } .reactionsMenu { - min-height: calc( - (var(--reaction-button-fontsize) + var(--cpd-separator-spacing)) * 2 - ); margin: 0; padding: 0; flex-grow: 1; @@ -84,7 +81,7 @@ div.reactionPopupMenuRoot.reactionPopupMenuModal > div > div { .verticalSeperator { background-color: var(--cpd-color-gray-800); - width: 1px; + width: 2px; height: auto; margin-left: var(--cpd-separator-spacing); margin-right: var(--cpd-separator-spacing); diff --git a/src/button/ReactionToggleButton.tsx b/src/button/ReactionToggleButton.tsx index 47ebe674..1c049497 100644 --- a/src/button/ReactionToggleButton.tsx +++ b/src/button/ReactionToggleButton.tsx @@ -177,10 +177,6 @@ export function ReactionToggleButton({ setErrorText(undefined); }, [showReactionsMenu]); - useEffect(() => { - return () => setShowReactionsMenu(false); - }, [setShowReactionsMenu]); - const canReact = !reactions[userId]; const sendRelation = useCallback( diff --git a/src/button/__snapshots__/ReactionToggleButton.test.tsx.snap b/src/button/__snapshots__/ReactionToggleButton.test.tsx.snap index 1950d566..604a615a 100644 --- a/src/button/__snapshots__/ReactionToggleButton.test.tsx.snap +++ b/src/button/__snapshots__/ReactionToggleButton.test.tsx.snap @@ -10,7 +10,7 @@ exports[`Can close search 1`] = ` aria-expanded="true" aria-haspopup="true" aria-label="action.raise_hand_or_send_reaction" - aria-labelledby=":rad:" + aria-labelledby=":r9l:" class="_button_i91xf_17 _has-icon_i91xf_66 _icon-only_i91xf_59" data-kind="primary" data-size="lg" @@ -46,7 +46,7 @@ exports[`Can fully expand emoji picker 1`] = ` aria-expanded="true" aria-haspopup="true" aria-label="action.raise_hand_or_send_reaction" - aria-labelledby=":r74:" + aria-labelledby=":r6c:" class="_button_i91xf_17 _has-icon_i91xf_66 _icon-only_i91xf_59" data-kind="primary" data-size="lg" @@ -79,7 +79,7 @@ exports[`Can lower hand 1`] = ` aria-expanded="false" aria-haspopup="true" aria-label="action.raise_hand_or_send_reaction" - aria-labelledby=":r3i:" + aria-labelledby=":r36:" class="_button_i91xf_17 raisedButton _has-icon_i91xf_66 _icon-only_i91xf_59" data-kind="primary" data-size="lg" @@ -145,7 +145,7 @@ exports[`Can raise hand 1`] = ` aria-expanded="false" aria-haspopup="true" aria-label="action.raise_hand_or_send_reaction" - aria-labelledby=":r1p:" + aria-labelledby=":r1j:" class="_button_i91xf_17 _has-icon_i91xf_66 _icon-only_i91xf_59" data-kind="secondary" data-size="lg"