Skip to content

Fix plugin sidebar panel rerender issues#1279

Merged
loiswells97 merged 2 commits into
mainfrom
lois/970-fix-plugin-panel-rerender
Nov 19, 2025
Merged

Fix plugin sidebar panel rerender issues#1279
loiswells97 merged 2 commits into
mainfrom
lois/970-fix-plugin-panel-rerender

Conversation

@loiswells97

@loiswells97 loiswells97 commented Nov 19, 2025

Copy link
Copy Markdown
Contributor

@loiswells97
loiswells97 temporarily deployed to previews/1279/merge November 19, 2025 15:21 — with GitHub Actions Inactive

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR addresses rerendering issues with plugin sidebar panels by wrapping the plugin menu options mapping in a useMemo hook to memoize the result and prevent unnecessary recreations on every render.

Key changes:

  • Wrapped the pluginMenuOptions creation with useMemo to memoize the plugin menu options array
  • Added useMemo to the React imports in Sidebar.jsx

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
src/components/Menus/Sidebar/Sidebar.jsx Wrapped plugin menu options mapping in useMemo to prevent unnecessary recreations
CHANGELOG.md Documented the fix for plugin sidebar panel rerendering issue

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread src/components/Menus/Sidebar/Sidebar.jsx
@loiswells97
loiswells97 merged commit 9a0e920 into main Nov 19, 2025
13 of 14 checks passed
@loiswells97
loiswells97 deleted the lois/970-fix-plugin-panel-rerender branch November 19, 2025 16:00
@loiswells97 loiswells97 mentioned this pull request Dec 9, 2025
loiswells97 added a commit that referenced this pull request Dec 9, 2025
### Added

- Tertiary danger button styles (#1280)

### Fixed

- Plugin sidebar panel rerendering (#1279)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants