From 0a7aee9cdb6e7060e0a62ecb597a127e5c77fc97 Mon Sep 17 00:00:00 2001 From: Claire Date: Tue, 11 Mar 2025 16:36:31 +0100 Subject: [PATCH] [Glitch] Fix unnecessary rerenders in composer dropdown menu Port 725a68d273b971c47f717b0e120fa3143e1a7286 to glitch-soc Signed-off-by: Claire --- .../compose/components/action_bar.jsx | 46 ++++++++++--------- 1 file changed, 24 insertions(+), 22 deletions(-) diff --git a/app/javascript/flavours/glitch/features/compose/components/action_bar.jsx b/app/javascript/flavours/glitch/features/compose/components/action_bar.jsx index 43c2899373..e277e300f8 100644 --- a/app/javascript/flavours/glitch/features/compose/components/action_bar.jsx +++ b/app/javascript/flavours/glitch/features/compose/components/action_bar.jsx @@ -1,4 +1,4 @@ -import { useCallback } from 'react'; +import { useMemo } from 'react'; import { defineMessages, useIntl } from 'react-intl'; @@ -28,28 +28,30 @@ export const ActionBar = () => { const dispatch = useDispatch(); const intl = useIntl(); - const handleLogoutClick = useCallback(() => { - dispatch(openModal({ modalType: 'CONFIRM_LOG_OUT' })); - }, [dispatch]); + const menu = useMemo(() => { + const handleLogoutClick = () => { + dispatch(openModal({ modalType: 'CONFIRM_LOG_OUT' })); + }; - let menu = []; - - menu.push({ text: intl.formatMessage(messages.edit_profile), href: '/settings/profile' }); - menu.push({ text: intl.formatMessage(messages.preferences), href: '/settings/preferences' }); - menu.push({ text: intl.formatMessage(messages.pins), to: '/pinned' }); - menu.push(null); - menu.push({ text: intl.formatMessage(messages.follow_requests), to: '/follow_requests' }); - menu.push({ text: intl.formatMessage(messages.favourites), to: '/favourites' }); - menu.push({ text: intl.formatMessage(messages.bookmarks), to: '/bookmarks' }); - menu.push({ text: intl.formatMessage(messages.lists), to: '/lists' }); - menu.push({ text: intl.formatMessage(messages.followed_tags), to: '/followed_tags' }); - menu.push(null); - menu.push({ text: intl.formatMessage(messages.mutes), to: '/mutes' }); - menu.push({ text: intl.formatMessage(messages.blocks), to: '/blocks' }); - menu.push({ text: intl.formatMessage(messages.domain_blocks), to: '/domain_blocks' }); - menu.push({ text: intl.formatMessage(messages.filters), href: '/filters' }); - menu.push(null); - menu.push({ text: intl.formatMessage(messages.logout), action: handleLogoutClick }); + return ([ + { text: intl.formatMessage(messages.edit_profile), href: '/settings/profile' }, + { text: intl.formatMessage(messages.preferences), href: '/settings/preferences' }, + { text: intl.formatMessage(messages.pins), to: '/pinned' }, + null, + { text: intl.formatMessage(messages.follow_requests), to: '/follow_requests' }, + { text: intl.formatMessage(messages.favourites), to: '/favourites' }, + { text: intl.formatMessage(messages.bookmarks), to: '/bookmarks' }, + { text: intl.formatMessage(messages.lists), to: '/lists' }, + { text: intl.formatMessage(messages.followed_tags), to: '/followed_tags' }, + null, + { text: intl.formatMessage(messages.mutes), to: '/mutes' }, + { text: intl.formatMessage(messages.blocks), to: '/blocks' }, + { text: intl.formatMessage(messages.domain_blocks), to: '/domain_blocks' }, + { text: intl.formatMessage(messages.filters), href: '/filters' }, + null, + { text: intl.formatMessage(messages.logout), action: handleLogoutClick }, + ]); + }, [intl, dispatch]); return (