Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Deprecate textinput contrast #4991

Open
wants to merge 8 commits into
base: main
Choose a base branch
from
+8 −2
Open

Deprecate textinput contrast #4991

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
8 commits
Select commit Hold shift + click to select a range
1b454cd
Update TextInput.tsx
CleverLemming1337 Sep 19, 2024
7c4daae
Mark `contrast` as deprecated in TextInput.docs.json
CleverLemming1337 Sep 19, 2024
1d3f548
Merge pull request #1 from CleverLemming1337/patch-1
CleverLemming1337 Sep 19, 2024
5a57643
Merge pull request #2 from CleverLemming1337/patch-2
CleverLemming1337 Sep 19, 2024
8180f86
Deprecate `contrast` in TextInputWithTokens
CleverLemming1337 Sep 19, 2024
a983311
Add changeset
CleverLemming1337 Sep 19, 2024
7a1b9d2
Merge branch 'main' into deprecate-textinput-contrast
CleverLemming1337 Nov 21, 2024
0b71456
Merge branch 'main' into deprecate-textinput-contrast
francinelucca Dec 2, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
5 changes: 5 additions & 0 deletions .changeset/slow-boats-roll.md
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,5 @@
---
'@primer/react': minor
---

Deprecate `contrast` prop of TextInput and TextInputWithTokens
1 change: 1 addition & 0 deletions packages/react/src/TextInput/TextInput.docs.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -77,6 +77,7 @@
"name": "contrast",
"type": "boolean",
"defaultValue": "false",
"deprecated": true,
"description": "Changes background color to a higher contrast color"
},
{
Expand Down
2 changes: 1 addition & 1 deletion packages/react/src/TextInput/TextInput.tsx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -66,7 +66,6 @@ const TextInput = React.forwardRef<HTMLInputElement, TextInputProps>(
trailingAction,
block,
className,
contrast,
disabled,
loading,
loaderPosition = 'auto',
Expand All @@ -82,6 +81,7 @@ const TextInput = React.forwardRef<HTMLInputElement, TextInputProps>(
minWidth: minWidthProp,
maxWidth: maxWidthProp,
variant: variantProp,
contrast,
// end deprecated props
type = 'text',
required,
Expand Down
2 changes: 1 addition & 1 deletion packages/react/src/TextInputWithTokens/TextInputWithTokens.tsx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -79,7 +79,7 @@ function TextInputWithTokensInnerComponent<TokenComponentType extends AnyReactCo
trailingVisual: TrailingVisual,
loading,
loaderPosition = 'auto',
contrast,
contrast, // deprecated
Copy link
Member

Choose a reason for hiding this comment

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

I think we'd also want to add the @deprecated decorator to the contrast prop in the StyledTextInputBaseWrapperProps object.
Though I do see this is also used in Select in addition to TextInput,TextArea and TextInputWithTokens.
@mperrotti wondering if we're good to remove contrast for all of these or should we explore a different strategy? 🤔

className,
block,
disabled,
Expand Down
Loading