avoid resetting visible plugin state when explorer or history context changes (#2792)

thomasheyenbrock · web-flow · commit 36839800de12 · 2022-10-06T09:23:31.000+02:00
diff --git a/.changeset/sour-kangaroos-sing.md b/.changeset/sour-kangaroos-sing.md
@@ -0,0 +1,5 @@
+---
+'@graphiql/react': patch
+---
+
+Avoid resetting visible plugin state when explorer or history context changes
diff --git a/packages/graphiql-react/src/plugin.tsx b/packages/graphiql-react/src/plugin.tsx
@@ -97,15 +97,17 @@ export function PluginContextProvider(props: PluginContextProviderProps) {
   const explorerContext = useExplorerContext();
   const historyContext = useHistoryContext();
 
+  const hasExplorerContext = Boolean(explorerContext);
+  const hasHistoryContext = Boolean(historyContext);
   const plugins = useMemo(() => {
     const pluginList: GraphiQLPlugin[] = [];
     const pluginTitles: Record<string, true> = {};
 
-    if (explorerContext) {
+    if (hasExplorerContext) {
       pluginList.push(DOC_EXPLORER_PLUGIN);
       pluginTitles[DOC_EXPLORER_PLUGIN.title] = true;
     }
-    if (historyContext) {
+    if (hasHistoryContext) {
       pluginList.push(HISTORY_PLUGIN);
       pluginTitles[HISTORY_PLUGIN.title] = true;
     }
@@ -125,7 +127,7 @@ export function PluginContextProvider(props: PluginContextProviderProps) {
     }
 
     return pluginList;
-  }, [explorerContext, historyContext, props.plugins]);
+  }, [hasExplorerContext, hasHistoryContext, props.plugins]);
 
   const [visiblePlugin, internalSetVisiblePlugin] =
     useState<GraphiQLPlugin | null>(() => {

-Original file line number
+Diff line change
@@ @@ -0,0 +1,5 @@ @@
 +---
 +'@graphiql/react': patch
 +---
++
 +Avoid resetting visible plugin state when explorer or history context changes