web/communities: remove explanatory comments from sub-community dropdown

Author: kevinortiz43

app/web/features/communities/CommunityPage/CommunityPageSubHeader.tsx

@@ -36,7 +36,6 @@ const StyledBreadcrumbs = styled(Breadcrumbs)(({ theme }) => ({
     "& .MuiTypography-root, & .MuiLink-root, & .MuiButton-root": {
       fontSize: "inherit",
     },
-    // Wrap a whole crumb to the next line instead of truncating a name.
     "& li": {
       flexShrink: 0,
       whiteSpace: "nowrap",
@@ -56,8 +55,6 @@ export default function CommunityPageSubHeader({
 
   const router = useRouter();
 
-  // Read the children query here (not in the dropdown) so the whole breadcrumb element can be
-  // omitted on leaves — MUI Breadcrumbs renders a dangling separator for a child that returns null.
   const {
     data: subCommunitiesData,
     hasNextPage,

app/web/features/communities/CommunityPage/SubCommunitiesDropdown.test.tsx

@@ -16,9 +16,6 @@ jest.mock("next/router", () => ({
   }),
 }));
 
-// jest.mock("next/router") above replaces the whole module, which drops the `Router.events`
-// emitter @sentry/nextjs reads at import time - stub Sentry out the same way CommunitySearch's
-// test does so the import doesn't crash.
 jest.mock("platform/sentry", () => {
   const mockCaptureException = jest.fn();
   return {
@@ -123,7 +120,6 @@ describe("SubCommunitiesDropdown", () => {
     const searchInput = screen.getByPlaceholderText(
       t("communities:sub_community_search_placeholder"),
     );
-    // stopPropagation on the search input must NOT swallow Escape, or the menu can't close.
     searchInput.focus();
     await user.keyboard("{Escape}");
 

app/web/features/communities/CommunityPage/SubCommunitiesDropdown.tsx

@@ -60,7 +60,6 @@ export default function SubCommunitiesDropdown({
     handleClose();
   };
 
-  // MUI Menu hijacks typeahead/arrow keys meant for the search input; Escape must still bubble to close.
   const handleSearchKeyDown = (event: KeyboardEvent<HTMLInputElement>) => {
     if (event.key !== "Escape") {
       event.stopPropagation();
@@ -93,7 +92,6 @@ export default function SubCommunitiesDropdown({
         disableAutoFocusItem
         anchorOrigin={{ vertical: "bottom", horizontal: "left" }}
         transformOrigin={{ vertical: "top", horizontal: "left" }}
-        // fixed width so the Paper doesn't reflow as results filter
         slotProps={{
           paper: { sx: { width: 300, maxWidth: "90vw", maxHeight: 288 } },
         }}

app/web/features/communities/discussions/DiscussionPage.test.tsx

@@ -124,7 +124,6 @@ describe("Discussion page", () => {
   beforeEach(() => {
     getLiteUserMock.mockImplementation(getLiteUser);
     getCommunityMock.mockResolvedValue(community);
-    // CommunityPageSubHeader lists sub-communities for its breadcrumb drill-down dropdown.
     listCommunitiesMock.mockResolvedValue({
       communitiesList: [],
       nextPageToken: "",