From b4f3816f0c954f8d8e5c208e18ac4583a3846a6e Mon Sep 17 00:00:00 2001
From: Tanmay Vardhaman Thole <72058456+tanmaythole@users.noreply.github.com>
Date: Mon, 16 Oct 2023 20:34:31 +0530
Subject: [PATCH] migrate channel header dropdown comp to functional comp
 (#24793)

* migrate channel header comp to functional comp

* snapshots updated

---------

Co-authored-by: Mattermost Build <build@mattermost.com>
---
 .../channel_header.test.tsx.snap              | 32 +++++++++----------
 .../channel_header_dropdown.tsx               | 24 +++++++-------
 2 files changed, 27 insertions(+), 29 deletions(-)

diff --git a/webapp/channels/src/components/channel_header/__snapshots__/channel_header.test.tsx.snap b/webapp/channels/src/components/channel_header/__snapshots__/channel_header.test.tsx.snap
index def8946c69..0b0e66ea78 100644
--- a/webapp/channels/src/components/channel_header/__snapshots__/channel_header.test.tsx.snap
+++ b/webapp/channels/src/components/channel_header/__snapshots__/channel_header.test.tsx.snap
@@ -61,7 +61,7 @@ exports[`components/ChannelHeader should match snapshot with last active display
                   />
                 </button>
               </div>
-              <ChannelHeaderDropdown />
+              <Memo(ChannelHeaderDropdown) />
             </MenuWrapper>
             <OverlayTrigger
               defaultOverlayShown={false}
@@ -384,7 +384,7 @@ exports[`components/ChannelHeader should match snapshot with no last active disp
                   />
                 </button>
               </div>
-              <ChannelHeaderDropdown />
+              <Memo(ChannelHeaderDropdown) />
             </MenuWrapper>
             <OverlayTrigger
               defaultOverlayShown={false}
@@ -679,7 +679,7 @@ exports[`components/ChannelHeader should render active channel files 1`] = `
                   />
                 </button>
               </div>
-              <ChannelHeaderDropdown />
+              <Memo(ChannelHeaderDropdown) />
             </MenuWrapper>
             <OverlayTrigger
               defaultOverlayShown={false}
@@ -980,7 +980,7 @@ exports[`components/ChannelHeader should render active flagged posts 1`] = `
                   />
                 </button>
               </div>
-              <ChannelHeaderDropdown />
+              <Memo(ChannelHeaderDropdown) />
             </MenuWrapper>
             <OverlayTrigger
               defaultOverlayShown={false}
@@ -1281,7 +1281,7 @@ exports[`components/ChannelHeader should render active mentions posts 1`] = `
                   />
                 </button>
               </div>
-              <ChannelHeaderDropdown />
+              <Memo(ChannelHeaderDropdown) />
             </MenuWrapper>
             <OverlayTrigger
               defaultOverlayShown={false}
@@ -1582,7 +1582,7 @@ exports[`components/ChannelHeader should render active pinned posts 1`] = `
                   />
                 </button>
               </div>
-              <ChannelHeaderDropdown />
+              <Memo(ChannelHeaderDropdown) />
             </MenuWrapper>
             <OverlayTrigger
               defaultOverlayShown={false}
@@ -1886,7 +1886,7 @@ exports[`components/ChannelHeader should render archived view 1`] = `
                   />
                 </button>
               </div>
-              <ChannelHeaderDropdown />
+              <Memo(ChannelHeaderDropdown) />
             </MenuWrapper>
           </div>
         </div>
@@ -2153,7 +2153,7 @@ exports[`components/ChannelHeader should render correct menu when muted 1`] = `
                   />
                 </button>
               </div>
-              <ChannelHeaderDropdown />
+              <Memo(ChannelHeaderDropdown) />
             </MenuWrapper>
             <OverlayTrigger
               defaultOverlayShown={false}
@@ -2486,7 +2486,7 @@ exports[`components/ChannelHeader should render not active channel files 1`] = `
                   />
                 </button>
               </div>
-              <ChannelHeaderDropdown />
+              <Memo(ChannelHeaderDropdown) />
             </MenuWrapper>
             <OverlayTrigger
               defaultOverlayShown={false}
@@ -2795,7 +2795,7 @@ exports[`components/ChannelHeader should render properly when custom status is e
                   />
                 </button>
               </div>
-              <ChannelHeaderDropdown />
+              <Memo(ChannelHeaderDropdown) />
             </MenuWrapper>
             <OverlayTrigger
               defaultOverlayShown={false}
@@ -3118,7 +3118,7 @@ exports[`components/ChannelHeader should render properly when custom status is s
                   />
                 </button>
               </div>
-              <ChannelHeaderDropdown />
+              <Memo(ChannelHeaderDropdown) />
             </MenuWrapper>
             <OverlayTrigger
               defaultOverlayShown={false}
@@ -3453,7 +3453,7 @@ exports[`components/ChannelHeader should render properly when empty 1`] = `
                   />
                 </button>
               </div>
-              <ChannelHeaderDropdown />
+              <Memo(ChannelHeaderDropdown) />
             </MenuWrapper>
             <OverlayTrigger
               defaultOverlayShown={false}
@@ -3754,7 +3754,7 @@ exports[`components/ChannelHeader should render properly when populated 1`] = `
                   />
                 </button>
               </div>
-              <ChannelHeaderDropdown />
+              <Memo(ChannelHeaderDropdown) />
             </MenuWrapper>
             <OverlayTrigger
               defaultOverlayShown={false}
@@ -4055,7 +4055,7 @@ exports[`components/ChannelHeader should render properly when populated with cha
                   />
                 </button>
               </div>
-              <ChannelHeaderDropdown />
+              <Memo(ChannelHeaderDropdown) />
             </MenuWrapper>
             <OverlayTrigger
               defaultOverlayShown={false}
@@ -4387,7 +4387,7 @@ exports[`components/ChannelHeader should render shared view 1`] = `
                   />
                 </button>
               </div>
-              <ChannelHeaderDropdown />
+              <Memo(ChannelHeaderDropdown) />
             </MenuWrapper>
             <OverlayTrigger
               defaultOverlayShown={false}
@@ -4690,7 +4690,7 @@ exports[`components/ChannelHeader should render the pinned icon with the pinned
                   />
                 </button>
               </div>
-              <ChannelHeaderDropdown />
+              <Memo(ChannelHeaderDropdown) />
             </MenuWrapper>
             <OverlayTrigger
               defaultOverlayShown={false}
diff --git a/webapp/channels/src/components/channel_header_dropdown/channel_header_dropdown.tsx b/webapp/channels/src/components/channel_header_dropdown/channel_header_dropdown.tsx
index d1f61dfc8e..6e80d6bc30 100644
--- a/webapp/channels/src/components/channel_header_dropdown/channel_header_dropdown.tsx
+++ b/webapp/channels/src/components/channel_header_dropdown/channel_header_dropdown.tsx
@@ -1,22 +1,20 @@
 // Copyright (c) 2015-present Mattermost, Inc. All Rights Reserved.
 // See LICENSE.txt for license information.
 
-import React from 'react';
+import React, {memo} from 'react';
 
 import {ChannelHeaderDropdownItems} from 'components/channel_header_dropdown';
 import Menu from 'components/widgets/menu/menu';
 
 import {localizeMessage} from 'utils/utils';
 
-export default class ChannelHeaderDropdown extends React.PureComponent {
-    render() {
-        return (
-            <Menu
-                id='channelHeaderDropdownMenu'
-                ariaLabel={localizeMessage('channel_header.menuAriaLabel', 'Channel Menu').toLowerCase()}
-            >
-                <ChannelHeaderDropdownItems isMobile={false}/>
-            </Menu>
-        );
-    }
-}
+const ChannelHeaderDropdown = () => (
+    <Menu
+        id='channelHeaderDropdownMenu'
+        ariaLabel={localizeMessage('channel_header.menuAriaLabel', 'Channel Menu').toLowerCase()}
+    >
+        <ChannelHeaderDropdownItems isMobile={false}/>
+    </Menu>
+);
+
+export default memo(ChannelHeaderDropdown);