From a574534e3aa6181c3c3e126e51e32bd9363b0b60 Mon Sep 17 00:00:00 2001
From: Matheus Wichman <matheushw@outlook.com>
Date: Thu, 22 Apr 2021 10:28:29 -0300
Subject: [PATCH 1/2] [TextField] Make the position prop required in
 InputAdornment

---
 docs/pages/api-docs/input-adornment.json                 | 5 ++++-
 docs/src/pages/guides/migration-v4/migration-v4.md       | 9 +++++++++
 .../material-ui/src/InputAdornment/InputAdornment.d.ts   | 2 +-
 .../material-ui/src/InputAdornment/InputAdornment.js     | 2 +-
 4 files changed, 15 insertions(+), 3 deletions(-)

diff --git a/docs/pages/api-docs/input-adornment.json b/docs/pages/api-docs/input-adornment.json
index 1d8bdcd184e7be..18c5dd1610fcb4 100644
--- a/docs/pages/api-docs/input-adornment.json
+++ b/docs/pages/api-docs/input-adornment.json
@@ -1,11 +1,14 @@
 {
   "props": {
+    "position": {
+      "type": { "name": "enum", "description": "'end'<br>&#124;&nbsp;'start'" },
+      "required": true
+    },
     "children": { "type": { "name": "node" } },
     "classes": { "type": { "name": "object" } },
     "component": { "type": { "name": "elementType" } },
     "disablePointerEvents": { "type": { "name": "bool" } },
     "disableTypography": { "type": { "name": "bool" } },
-    "position": { "type": { "name": "enum", "description": "'end'<br>&#124;&nbsp;'start'" } },
     "sx": { "type": { "name": "object" } },
     "variant": {
       "type": {
diff --git a/docs/src/pages/guides/migration-v4/migration-v4.md b/docs/src/pages/guides/migration-v4/migration-v4.md
index 9dba9813dffef0..0caa54092b156c 100644
--- a/docs/src/pages/guides/migration-v4/migration-v4.md
+++ b/docs/src/pages/guides/migration-v4/migration-v4.md
@@ -1236,6 +1236,15 @@ As the core components use emotion as a styled engine, the props used by emotion
   +<Input size="small" />
   ```
 
+- Set the InputAdornment `position` prop to `start` or `end`. Use `start` if used as the value of the `startAdornment` prop. Use `end` if used as the value of the `endAdornment` prop.
+
+  ```diff
+  -<TextField endAdornment={<InputAdornment>Kg</InputAdornment>} />
+  +<TextField endAdornment={<InputAdornment position="end">Kg</InputAdornment>} />
+  -<TextField startAdornment={<InputAdornment>Kg</InputAdornment>} />
+  +<TextField startAdornment={<InputAdornment position="start">Kg</InputAdornment>} />
+  ```
+
 ### TextareaAutosize
 
 - Remove the `rows` prop, use the `minRows` prop instead.
diff --git a/packages/material-ui/src/InputAdornment/InputAdornment.d.ts b/packages/material-ui/src/InputAdornment/InputAdornment.d.ts
index 9feb695c427006..73ae86db7e452c 100644
--- a/packages/material-ui/src/InputAdornment/InputAdornment.d.ts
+++ b/packages/material-ui/src/InputAdornment/InputAdornment.d.ts
@@ -42,7 +42,7 @@ export interface InputAdornmentTypeMap<P = {}, D extends React.ElementType = 'di
     /**
      * The position this adornment should appear relative to the `Input`.
      */
-    position?: 'start' | 'end';
+    position: 'start' | 'end';
     /**
      * The system prop that allows defining system overrides as well as additional CSS styles.
      */
diff --git a/packages/material-ui/src/InputAdornment/InputAdornment.js b/packages/material-ui/src/InputAdornment/InputAdornment.js
index 82abfdd00868d8..d04b53e6cce747 100644
--- a/packages/material-ui/src/InputAdornment/InputAdornment.js
+++ b/packages/material-ui/src/InputAdornment/InputAdornment.js
@@ -180,7 +180,7 @@ InputAdornment.propTypes /* remove-proptypes */ = {
   /**
    * The position this adornment should appear relative to the `Input`.
    */
-  position: PropTypes.oneOf(['end', 'start']),
+  position: PropTypes.oneOf(['end', 'start']).isRequired,
   /**
    * The system prop that allows defining system overrides as well as additional CSS styles.
    */

From 2024c4a5bf2743e3bbfe6d6efdbfcbc7800ec4ff Mon Sep 17 00:00:00 2001
From: Matheus Wichman <matheushw@outlook.com>
Date: Thu, 22 Apr 2021 14:56:41 -0300
Subject: [PATCH 2/2] group changes

---
 docs/src/pages/guides/migration-v4/migration-v4.md | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/docs/src/pages/guides/migration-v4/migration-v4.md b/docs/src/pages/guides/migration-v4/migration-v4.md
index 0caa54092b156c..852debe0ed4151 100644
--- a/docs/src/pages/guides/migration-v4/migration-v4.md
+++ b/docs/src/pages/guides/migration-v4/migration-v4.md
@@ -1239,10 +1239,10 @@ As the core components use emotion as a styled engine, the props used by emotion
 - Set the InputAdornment `position` prop to `start` or `end`. Use `start` if used as the value of the `startAdornment` prop. Use `end` if used as the value of the `endAdornment` prop.
 
   ```diff
-  -<TextField endAdornment={<InputAdornment>Kg</InputAdornment>} />
-  +<TextField endAdornment={<InputAdornment position="end">Kg</InputAdornment>} />
   -<TextField startAdornment={<InputAdornment>Kg</InputAdornment>} />
+  -<TextField endAdornment={<InputAdornment>Kg</InputAdornment>} />
   +<TextField startAdornment={<InputAdornment position="start">Kg</InputAdornment>} />
+  +<TextField endAdornment={<InputAdornment position="end">Kg</InputAdornment>} />
   ```
 
 ### TextareaAutosize