Skip to content

Commit dc06b14

Browse files
Remove border: none from TextInput internal styles (#1606)
1 parent 0fbed42 commit dc06b14

File tree

3 files changed

+7
-10
lines changed

3 files changed

+7
-10
lines changed

src/file-picker/__tests__/__snapshots__/FilePicker.test.js.snap

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,7 @@ exports[`<FilePicker /> snapshot - multiple files selected 1`] = `
1212
/>
1313
<input
1414
aria-invalid="false"
15-
class="evergreen-file-picker-text-input ub-w_280px ub-fnt-fam_b77syt ub-b-btm_none ub-b-lft_none ub-b-rgt_none ub-b-top_none ub-otln_iu2jf4 ub-txt-deco_none ub-b-btm-clr_d8dae5 ub-b-lft-clr_d8dae5 ub-b-rgt-clr_d8dae5 ub-b-top-clr_d8dae5 ub-bblr_4px ub-bbrr_0-important ub-btlr_4px ub-btrr_0-important ub-b-btm-stl_solid ub-b-lft-stl_solid ub-b-rgt-stl_solid ub-b-top-stl_solid ub-b-btm-wdt_1px ub-b-lft-wdt_1px ub-b-rgt-wdt_1px ub-b-top-wdt_1px ub-color_474d66 ub-fnt-sze_12px ub-ln-ht_16px ub-pl_12px ub-pr_12px ub-tstn_n1akt6 ub-h_32px ub-bg-clr_white ub-color_8f95b2_1b2gqua ub-crsr_not-allowed_10nxvc1 ub-bg-clr_F4F5F9_10nxvc1 ub-color_696f8c_10nxvc1 ub-z-idx_2_1bcl2f4 ub-bs_13po50p_1bcl2f4 ub-b-btm-clr_ADC2FF_1bcl2f4 ub-b-lft-clr_ADC2FF_1bcl2f4 ub-b-rgt-clr_ADC2FF_1bcl2f4 ub-b-top-clr_ADC2FF_1bcl2f4 ub-b-btm-clr_D14343_fpx4do ub-b-lft-clr_D14343_fpx4do ub-b-rgt-clr_D14343_fpx4do ub-b-top-clr_D14343_fpx4do ub-flx_1 ub-txt-ovrf_ellipsis ub-f-wght_400 ub-ltr-spc_0 ub-box-szg_border-box"
15+
class="evergreen-file-picker-text-input ub-w_280px ub-fnt-fam_b77syt ub-otln_iu2jf4 ub-txt-deco_none ub-b-btm-clr_d8dae5 ub-b-lft-clr_d8dae5 ub-b-rgt-clr_d8dae5 ub-b-top-clr_d8dae5 ub-bblr_4px ub-bbrr_0-important ub-btlr_4px ub-btrr_0-important ub-b-btm-stl_solid ub-b-lft-stl_solid ub-b-rgt-stl_solid ub-b-top-stl_solid ub-b-btm-wdt_1px ub-b-lft-wdt_1px ub-b-rgt-wdt_1px ub-b-top-wdt_1px ub-color_474d66 ub-fnt-sze_12px ub-ln-ht_16px ub-pl_12px ub-pr_12px ub-tstn_n1akt6 ub-h_32px ub-bg-clr_white ub-color_8f95b2_1b2gqua ub-crsr_not-allowed_10nxvc1 ub-bg-clr_F4F5F9_10nxvc1 ub-color_696f8c_10nxvc1 ub-z-idx_2_1bcl2f4 ub-bs_13po50p_1bcl2f4 ub-b-btm-clr_ADC2FF_1bcl2f4 ub-b-lft-clr_ADC2FF_1bcl2f4 ub-b-rgt-clr_ADC2FF_1bcl2f4 ub-b-top-clr_ADC2FF_1bcl2f4 ub-b-btm-clr_D14343_fpx4do ub-b-lft-clr_D14343_fpx4do ub-b-rgt-clr_D14343_fpx4do ub-b-top-clr_D14343_fpx4do ub-flx_1 ub-txt-ovrf_ellipsis ub-f-wght_400 ub-ltr-spc_0 ub-box-szg_border-box"
1616
placeholder="Select a file to upload…"
1717
readonly=""
1818
spellcheck="true"
@@ -40,7 +40,7 @@ exports[`<FilePicker /> snapshot - no file selected 1`] = `
4040
/>
4141
<input
4242
aria-invalid="false"
43-
class="evergreen-file-picker-text-input ub-w_280px ub-fnt-fam_b77syt ub-b-btm_none ub-b-lft_none ub-b-rgt_none ub-b-top_none ub-otln_iu2jf4 ub-txt-deco_none ub-b-btm-clr_d8dae5 ub-b-lft-clr_d8dae5 ub-b-rgt-clr_d8dae5 ub-b-top-clr_d8dae5 ub-bblr_4px ub-bbrr_0-important ub-btlr_4px ub-btrr_0-important ub-b-btm-stl_solid ub-b-lft-stl_solid ub-b-rgt-stl_solid ub-b-top-stl_solid ub-b-btm-wdt_1px ub-b-lft-wdt_1px ub-b-rgt-wdt_1px ub-b-top-wdt_1px ub-color_474d66 ub-fnt-sze_12px ub-ln-ht_16px ub-pl_12px ub-pr_12px ub-tstn_n1akt6 ub-h_32px ub-bg-clr_white ub-color_8f95b2_1b2gqua ub-crsr_not-allowed_10nxvc1 ub-bg-clr_F4F5F9_10nxvc1 ub-color_696f8c_10nxvc1 ub-z-idx_2_1bcl2f4 ub-bs_13po50p_1bcl2f4 ub-b-btm-clr_ADC2FF_1bcl2f4 ub-b-lft-clr_ADC2FF_1bcl2f4 ub-b-rgt-clr_ADC2FF_1bcl2f4 ub-b-top-clr_ADC2FF_1bcl2f4 ub-b-btm-clr_D14343_fpx4do ub-b-lft-clr_D14343_fpx4do ub-b-rgt-clr_D14343_fpx4do ub-b-top-clr_D14343_fpx4do ub-flx_1 ub-txt-ovrf_ellipsis ub-f-wght_400 ub-ltr-spc_0 ub-box-szg_border-box"
43+
class="evergreen-file-picker-text-input ub-w_280px ub-fnt-fam_b77syt ub-otln_iu2jf4 ub-txt-deco_none ub-b-btm-clr_d8dae5 ub-b-lft-clr_d8dae5 ub-b-rgt-clr_d8dae5 ub-b-top-clr_d8dae5 ub-bblr_4px ub-bbrr_0-important ub-btlr_4px ub-btrr_0-important ub-b-btm-stl_solid ub-b-lft-stl_solid ub-b-rgt-stl_solid ub-b-top-stl_solid ub-b-btm-wdt_1px ub-b-lft-wdt_1px ub-b-rgt-wdt_1px ub-b-top-wdt_1px ub-color_474d66 ub-fnt-sze_12px ub-ln-ht_16px ub-pl_12px ub-pr_12px ub-tstn_n1akt6 ub-h_32px ub-bg-clr_white ub-color_8f95b2_1b2gqua ub-crsr_not-allowed_10nxvc1 ub-bg-clr_F4F5F9_10nxvc1 ub-color_696f8c_10nxvc1 ub-z-idx_2_1bcl2f4 ub-bs_13po50p_1bcl2f4 ub-b-btm-clr_ADC2FF_1bcl2f4 ub-b-lft-clr_ADC2FF_1bcl2f4 ub-b-rgt-clr_ADC2FF_1bcl2f4 ub-b-top-clr_ADC2FF_1bcl2f4 ub-b-btm-clr_D14343_fpx4do ub-b-lft-clr_D14343_fpx4do ub-b-rgt-clr_D14343_fpx4do ub-b-top-clr_D14343_fpx4do ub-flx_1 ub-txt-ovrf_ellipsis ub-f-wght_400 ub-ltr-spc_0 ub-box-szg_border-box"
4444
placeholder="Select a file to upload…"
4545
readonly=""
4646
spellcheck="true"
@@ -68,7 +68,7 @@ exports[`<FilePicker /> snapshot - one file seleted 1`] = `
6868
/>
6969
<input
7070
aria-invalid="false"
71-
class="evergreen-file-picker-text-input ub-w_280px ub-fnt-fam_b77syt ub-b-btm_none ub-b-lft_none ub-b-rgt_none ub-b-top_none ub-otln_iu2jf4 ub-txt-deco_none ub-b-btm-clr_d8dae5 ub-b-lft-clr_d8dae5 ub-b-rgt-clr_d8dae5 ub-b-top-clr_d8dae5 ub-bblr_4px ub-bbrr_0-important ub-btlr_4px ub-btrr_0-important ub-b-btm-stl_solid ub-b-lft-stl_solid ub-b-rgt-stl_solid ub-b-top-stl_solid ub-b-btm-wdt_1px ub-b-lft-wdt_1px ub-b-rgt-wdt_1px ub-b-top-wdt_1px ub-color_474d66 ub-fnt-sze_12px ub-ln-ht_16px ub-pl_12px ub-pr_12px ub-tstn_n1akt6 ub-h_32px ub-bg-clr_white ub-color_8f95b2_1b2gqua ub-crsr_not-allowed_10nxvc1 ub-bg-clr_F4F5F9_10nxvc1 ub-color_696f8c_10nxvc1 ub-z-idx_2_1bcl2f4 ub-bs_13po50p_1bcl2f4 ub-b-btm-clr_ADC2FF_1bcl2f4 ub-b-lft-clr_ADC2FF_1bcl2f4 ub-b-rgt-clr_ADC2FF_1bcl2f4 ub-b-top-clr_ADC2FF_1bcl2f4 ub-b-btm-clr_D14343_fpx4do ub-b-lft-clr_D14343_fpx4do ub-b-rgt-clr_D14343_fpx4do ub-b-top-clr_D14343_fpx4do ub-flx_1 ub-txt-ovrf_ellipsis ub-f-wght_400 ub-ltr-spc_0 ub-box-szg_border-box"
71+
class="evergreen-file-picker-text-input ub-w_280px ub-fnt-fam_b77syt ub-otln_iu2jf4 ub-txt-deco_none ub-b-btm-clr_d8dae5 ub-b-lft-clr_d8dae5 ub-b-rgt-clr_d8dae5 ub-b-top-clr_d8dae5 ub-bblr_4px ub-bbrr_0-important ub-btlr_4px ub-btrr_0-important ub-b-btm-stl_solid ub-b-lft-stl_solid ub-b-rgt-stl_solid ub-b-top-stl_solid ub-b-btm-wdt_1px ub-b-lft-wdt_1px ub-b-rgt-wdt_1px ub-b-top-wdt_1px ub-color_474d66 ub-fnt-sze_12px ub-ln-ht_16px ub-pl_12px ub-pr_12px ub-tstn_n1akt6 ub-h_32px ub-bg-clr_white ub-color_8f95b2_1b2gqua ub-crsr_not-allowed_10nxvc1 ub-bg-clr_F4F5F9_10nxvc1 ub-color_696f8c_10nxvc1 ub-z-idx_2_1bcl2f4 ub-bs_13po50p_1bcl2f4 ub-b-btm-clr_ADC2FF_1bcl2f4 ub-b-lft-clr_ADC2FF_1bcl2f4 ub-b-rgt-clr_ADC2FF_1bcl2f4 ub-b-top-clr_ADC2FF_1bcl2f4 ub-b-btm-clr_D14343_fpx4do ub-b-lft-clr_D14343_fpx4do ub-b-rgt-clr_D14343_fpx4do ub-b-top-clr_D14343_fpx4do ub-flx_1 ub-txt-ovrf_ellipsis ub-f-wght_400 ub-ltr-spc_0 ub-box-szg_border-box"
7272
placeholder="Select a file to upload…"
7373
readonly=""
7474
spellcheck="true"

src/text-input/__tests__/TextInput.test.js

Lines changed: 4 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -90,13 +90,11 @@ describe('TextInput', () => {
9090
it('should render with transparent border when appearance is none', () => {
9191
render(makeTextInputFixture({ appearance: 'none' }))
9292

93-
// For some reason we were applying a border: 1px solid transparent style and then overriding it with
94-
// the individual borderColor style, see https://github.com/segmentio/evergreen/issues/1581
9593
expect(screen.getByTestId('input')).toHaveStyle({
96-
borderTop: '1px solid transparent',
97-
borderBottom: '1px solid transparent',
98-
borderLeft: '1px solid transparent',
99-
borderRight: '1px solid transparent'
94+
borderTopColor: 'transparent',
95+
borderBottomColor: 'transparent',
96+
borderLeftColor: 'transparent',
97+
borderRightColor: 'transparent'
10098
})
10199
})
102100
})

src/text-input/src/TextInput.js

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,6 @@ const pseudoSelectors = {
1515
}
1616

1717
const internalStyles = {
18-
border: 'none',
1918
MozAppearance: 'none',
2019
outline: 'none',
2120
textDecoration: 'none',

0 commit comments

Comments
 (0)