From 5d43e3880ae36365333cb16068b6886dcb9a90c4 Mon Sep 17 00:00:00 2001
From: eps1lon <sebastian.silbermann@vercel.com>
Date: Fri, 17 May 2024 12:02:50 -0700
Subject: [PATCH] Revert "popoverTarget -> popoverTargetElement"

This reverts commit b394c001c7544b366820380952c281b77c90eab9.
---
 .../AttributeTableSnapshot.md                 | 31 ++-----------------
 fixtures/attribute-behavior/src/attributes.js |  5 ---
 .../src/client/ReactDOMComponent.js           |  2 --
 .../src/shared/getAttributeAlias.js           |  1 -
 .../src/shared/possibleStandardNames.js       |  3 +-
 .../src/__tests__/ReactDOMComponent-test.js   | 19 ------------
 6 files changed, 4 insertions(+), 57 deletions(-)

diff --git a/fixtures/attribute-behavior/AttributeTableSnapshot.md b/fixtures/attribute-behavior/AttributeTableSnapshot.md
index 427036879e1e8..112bf4ebfca63 100644
--- a/fixtures/attribute-behavior/AttributeTableSnapshot.md
+++ b/fixtures/attribute-behavior/AttributeTableSnapshot.md
@@ -8476,7 +8476,7 @@
 ## `popoverTarget` (on `<button>` inside `<div>`)
 | Test Case | Flags | Result |
 | --- | --- | --- |
-| `popoverTarget=(string)`| (initial, warning)| `<null>` |
+| `popoverTarget=(string)`| (initial)| `<null>` |
 | `popoverTarget=(empty string)`| (initial)| `<null>` |
 | `popoverTarget=(array with string)`| (initial)| `<null>` |
 | `popoverTarget=(empty array)`| (initial)| `<null>` |
@@ -8498,31 +8498,6 @@
 | `popoverTarget=(null)`| (initial)| `<null>` |
 | `popoverTarget=(undefined)`| (initial)| `<null>` |
 
-## `popoverTargetElement` (on `<button>` inside `<div>`)
-| Test Case | Flags | Result |
-| --- | --- | --- |
-| `popoverTargetElement=(string)`| (initial)| `<null>` |
-| `popoverTargetElement=(empty string)`| (initial)| `<null>` |
-| `popoverTargetElement=(array with string)`| (initial)| `<null>` |
-| `popoverTargetElement=(empty array)`| (initial)| `<null>` |
-| `popoverTargetElement=(object)`| (initial)| `<null>` |
-| `popoverTargetElement=(numeric string)`| (initial)| `<null>` |
-| `popoverTargetElement=(-1)`| (initial)| `<null>` |
-| `popoverTargetElement=(0)`| (initial)| `<null>` |
-| `popoverTargetElement=(integer)`| (initial)| `<null>` |
-| `popoverTargetElement=(NaN)`| (initial, warning)| `<null>` |
-| `popoverTargetElement=(float)`| (initial)| `<null>` |
-| `popoverTarget=(true)`| (initial, warning)| `<null>` |
-| `popoverTargetElement=(false)`| (initial, warning)| `<null>` |
-| `popoverTargetElement=(string 'true')`| (initial)| `<null>` |
-| `popoverTargetElement=(string 'false')`| (initial)| `<null>` |
-| `popoverTargetElement=(string 'on')`| (initial)| `<null>` |
-| `popoverTargetElement=(string 'off')`| (initial)| `<null>` |
-| `popoverTargetElement=(symbol)`| (initial, warning)| `<null>` |
-| `popoverTargetElement=(function)`| (initial, warning)| `<null>` |
-| `popoverTargetElement=(null)`| (initial)| `<null>` |
-| `popoverTargetElement=(undefined)`| (initial)| `<null>` |
-
 ## `popoverTargetAction` (on `<button>` inside `<div>`)
 | Test Case | Flags | Result |
 | --- | --- | --- |
@@ -8537,8 +8512,8 @@
 | `popoverTargetAction=(integer)`| (initial)| `"toggle"` |
 | `popoverTargetAction=(NaN)`| (initial, warning)| `"toggle"` |
 | `popoverTargetAction=(float)`| (initial)| `"toggle"` |
-| `popoverTargetAction=(true)`| (initial)| `"toggle"` |
-| `popoverTargetAction=(false)`| (initial)| `"toggle"` |
+| `popoverTargetAction=(true)`| (initial, warning)| `"toggle"` |
+| `popoverTargetAction=(false)`| (initial, warning)| `"toggle"` |
 | `popoverTargetAction=(string 'true')`| (initial)| `"toggle"` |
 | `popoverTargetAction=(string 'false')`| (initial)| `"toggle"` |
 | `popoverTargetAction=(string 'on')`| (initial)| `"toggle"` |
diff --git a/fixtures/attribute-behavior/src/attributes.js b/fixtures/attribute-behavior/src/attributes.js
index 66fbc96405f48..12faa69754fc1 100644
--- a/fixtures/attribute-behavior/src/attributes.js
+++ b/fixtures/attribute-behavior/src/attributes.js
@@ -1453,11 +1453,6 @@ const attributes = [
     read: element => element.popoverTargetElement,
     tagName: 'button',
   },
-  {
-    name: 'popoverTargetElement',
-    read: element => element.popoverTargetElement,
-    tagName: 'button',
-  },
   {name: 'popoverTargetAction', overrideStringValue: 'show', tagName: 'button'},
   {
     name: 'poster',
diff --git a/packages/react-dom-bindings/src/client/ReactDOMComponent.js b/packages/react-dom-bindings/src/client/ReactDOMComponent.js
index ecd7decb2c5d3..2a26580225365 100644
--- a/packages/react-dom-bindings/src/client/ReactDOMComponent.js
+++ b/packages/react-dom-bindings/src/client/ReactDOMComponent.js
@@ -1815,8 +1815,6 @@ function getPropNameFromAttributeName(attrName: string): string {
       return 'className';
     case 'for':
       return 'htmlFor';
-    case 'popopverTarget':
-      return 'popopverTargetElement';
     // TODO: The rest of the aliases.
     default:
       return attrName;
diff --git a/packages/react-dom-bindings/src/shared/getAttributeAlias.js b/packages/react-dom-bindings/src/shared/getAttributeAlias.js
index bbc47d53d5e4a..2ed432f20275b 100644
--- a/packages/react-dom-bindings/src/shared/getAttributeAlias.js
+++ b/packages/react-dom-bindings/src/shared/getAttributeAlias.js
@@ -10,7 +10,6 @@
 const aliases = new Map([
   ['acceptCharset', 'accept-charset'],
   ['htmlFor', 'for'],
-  ['popoverTargetElement', 'popoverTarget'],
   ['httpEquiv', 'http-equiv'],
   // HTML and SVG attributes, but the SVG attribute is case sensitive.],
   ['crossOrigin', 'crossorigin'],
diff --git a/packages/react-dom-bindings/src/shared/possibleStandardNames.js b/packages/react-dom-bindings/src/shared/possibleStandardNames.js
index ce3cf3ef36a6c..369b87110b235 100644
--- a/packages/react-dom-bindings/src/shared/possibleStandardNames.js
+++ b/packages/react-dom-bindings/src/shared/possibleStandardNames.js
@@ -345,9 +345,8 @@ const possibleStandardNames = {
   pointsaty: 'pointsAtY',
   pointsatz: 'pointsAtZ',
   popover: 'popover',
-  popovertarget: 'popoverTargetElement',
+  popovertarget: 'popoverTarget',
   popovertargetaction: 'popoverTargetAction',
-  popovertargetelement: 'popoverTargetElement',
   prefix: 'prefix',
   preservealpha: 'preserveAlpha',
   preserveaspectratio: 'preserveAspectRatio',
diff --git a/packages/react-dom/src/__tests__/ReactDOMComponent-test.js b/packages/react-dom/src/__tests__/ReactDOMComponent-test.js
index 51b79a904792b..b72b3da8ced02 100644
--- a/packages/react-dom/src/__tests__/ReactDOMComponent-test.js
+++ b/packages/react-dom/src/__tests__/ReactDOMComponent-test.js
@@ -2926,18 +2926,6 @@ describe('ReactDOMComponent', () => {
       }).toErrorDev(
         'Warning: Invalid DOM property `autofocus`. Did you mean `autoFocus`?\n    in input',
       );
-
-      await expect(async () => {
-        const container = document.createElement('div');
-        const root = ReactDOMClient.createRoot(container);
-        await act(() => {
-          root.render(
-            React.createElement('button', {popoverTarget: 'some-element'}),
-          );
-        });
-      }).toErrorDev(
-        'Warning: Invalid DOM property `popoverTarget`. Did you mean `popoverTargetElement`?\n    in button',
-      );
     });
 
     it('should suggest property name if available (ssr)', () => {
@@ -2955,13 +2943,6 @@ describe('ReactDOMComponent', () => {
       ).toErrorDev(
         'Warning: Invalid DOM property `autofocus`. Did you mean `autoFocus`?\n    in input',
       );
-      expect(() =>
-        ReactDOMServer.renderToString(
-          React.createElement('button', {popoverTarget: 'some-element'}),
-        ),
-      ).toErrorDev(
-        'Warning: Invalid DOM property `popoverTarget`. Did you mean `popoverTargetElement`?\n    in button',
-      );
     });
   });