-
Notifications
You must be signed in to change notification settings - Fork 393
change: [M3-9432] - Theme changes to InputBase (DebounceSearchTextField, Autocomplete, TextField) #11807
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.
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
change: [M3-9432] - Theme changes to InputBase (DebounceSearchTextField, Autocomplete, TextField) #11807
Changes from all commits
1f3c6a8
76ddcfd
9f9a1b2
ac15511
0a615aa
a3aaf05
7f986ae
c2a3d37
53b461b
8d3d3f2
9081c35
5f95e68
9314d83
a11226b
371707e
bba9dbe
13ae152
796a4bf
a1520d3
09b5f02
63bfbdb
c734c98
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,5 @@ | ||
| --- | ||
| "@linode/manager": Changed | ||
| --- | ||
|
|
||
| Enhance MUI Adornments: Unify Theme for Autocomplete and TextField Components via InputBase Styling ([#11807](https://github.com/linode/manager/pull/11807)) |
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,4 +1,11 @@ | ||
| import { Autocomplete, Box, Notice, Stack, Typography } from '@linode/ui'; | ||
| import { | ||
| Autocomplete, | ||
| Box, | ||
| InputAdornment, | ||
| Notice, | ||
| Stack, | ||
| Typography, | ||
| } from '@linode/ui'; | ||
| import { DateTime } from 'luxon'; | ||
| import React, { useMemo } from 'react'; | ||
|
|
||
|
|
@@ -178,15 +185,15 @@ export const ImageSelect = (props: Props) => { | |
| InputProps: { | ||
| startAdornment: | ||
| !multiple && value && !Array.isArray(value) ? ( | ||
| <OSIcon | ||
| fontSize="24px" | ||
| height="24px" | ||
| os={value.vendor ?? ''} | ||
| pl={1} | ||
| position="relative" | ||
| pr={2} | ||
| top={1} | ||
| /> | ||
| <InputAdornment position="start"> | ||
|
Contributor
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. When using adornments, let's ensure we use
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Should we create a ticket for this? Found several adornments not wrapped in
|
||
| <OSIcon | ||
| fontSize="20px" | ||
| height="20px" | ||
| os={value.vendor ?? ''} | ||
| position="relative" | ||
| top={1} | ||
| /> | ||
| </InputAdornment> | ||
| ) : null, | ||
| }, | ||
| }} | ||
|
|
||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -42,7 +42,6 @@ const useStyles = makeStyles()((theme: Theme) => ({ | |
| [theme.breakpoints.down('sm')]: { | ||
| textAlign: 'center', | ||
| }, | ||
| top: 2, | ||
|
Contributor
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Removed since |
||
| }, | ||
| })); | ||
|
|
||
|
|
@@ -53,15 +52,14 @@ export const StrengthIndicator = (props: Props) => { | |
|
|
||
| return ( | ||
| <Grid | ||
| className={classes.root} | ||
| container | ||
| data-qa-strength={strength} | ||
| spacing={1} | ||
| sx={{ | ||
| alignItems: 'flex-end', | ||
| alignItems: 'center', | ||
| paddingLeft: 0, | ||
| paddingRight: 0, | ||
| }} | ||
| className={classes.root} | ||
| container | ||
| data-qa-strength={strength} | ||
| > | ||
| {Array.from(Array(3), (v, idx) => idx + 1).map((idx) => ( | ||
| <Grid className={classes.blockOuter} key={idx} size={3}> | ||
|
|
@@ -76,7 +74,7 @@ export const StrengthIndicator = (props: Props) => { | |
| /> | ||
| </Grid> | ||
| ))} | ||
| <Grid className="py0" size={3}> | ||
| <Grid paddingLeft={1} size={3}> | ||
| <Typography | ||
| className={classes.strengthText} | ||
| data-qa-password-strength | ||
|
|
||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Deprecated - using
slotProps