diff --git a/package-lock.json b/package-lock.json index 4880e1c5..2ab69240 100644 --- a/package-lock.json +++ b/package-lock.json @@ -3828,9 +3828,9 @@ } }, "@edx/paragon": { - "version": "13.16.1", - "resolved": "https://registry.npmjs.org/@edx/paragon/-/paragon-13.16.1.tgz", - "integrity": "sha512-8GxeS0OQLKJKrkqUa6HRk+wi/96IvhuY6JjjQYIf4+SA6TXLu57SwvtsQN6sLuB3PTxnW64WQF3MepvpNo9VpQ==", + "version": "13.17.3", + "resolved": "https://registry.npmjs.org/@edx/paragon/-/paragon-13.17.3.tgz", + "integrity": "sha512-fUjrfNmeWIpEsroK0JuajIBHHh0BIvZTnBusTRqzvl5fFivNuhEdcG33oEZSVvfyRYtCgtnWmSRbvN5vGhjK6g==", "requires": { "@fortawesome/fontawesome-svg-core": "^1.2.30", "@fortawesome/free-solid-svg-icons": "^5.14.0", @@ -21284,9 +21284,9 @@ } }, "react-popper": { - "version": "2.2.4", - "resolved": "https://registry.npmjs.org/react-popper/-/react-popper-2.2.4.tgz", - "integrity": "sha512-NacOu4zWupdQjVXq02XpTD3yFPSfg5a7fex0wa3uGKVkFK7UN6LvVxgcb+xYr56UCuWiNPMH20tntdVdJRwYew==", + "version": "2.2.5", + "resolved": "https://registry.npmjs.org/react-popper/-/react-popper-2.2.5.tgz", + "integrity": "sha512-kxGkS80eQGtLl18+uig1UIf9MKixFSyPxglsgLBxlYnyDf65BiY9B3nZSc6C9XUNDgStROB0fMQlTEz1KxGddw==", "requires": { "react-fast-compare": "^3.0.1", "warning": "^4.0.2" diff --git a/package.json b/package.json index 7ff43b28..f5ca99d7 100644 --- a/package.json +++ b/package.json @@ -38,7 +38,7 @@ "@edx/frontend-component-cookie-policy-banner": "2.1.11", "@edx/frontend-component-header": "2.2.4", "@edx/frontend-platform": "1.8.4", - "@edx/paragon": "13.16.1", + "@edx/paragon": "13.17.3", "@fortawesome/fontawesome-svg-core": "1.2.32", "@fortawesome/free-brands-svg-icons": "5.15.1", "@fortawesome/free-regular-svg-icons": "5.15.1", diff --git a/src/common-components/AuthnValidationFormGroup.jsx b/src/common-components/AuthnValidationFormGroup.jsx index 97f9c4ae..2cd1ebf7 100644 --- a/src/common-components/AuthnValidationFormGroup.jsx +++ b/src/common-components/AuthnValidationFormGroup.jsx @@ -61,7 +61,6 @@ const AuthnCustomValidationFormGroup = (props) => { id: props.for, type: props.type, value: props.value, - className: props.inputFieldStyle, 'aria-invalid': props.ariaInvalid, }; inputProps.onChange = (e) => onChangeHandler(e, onChange); @@ -71,7 +70,7 @@ const AuthnCustomValidationFormGroup = (props) => { if (props.type === 'select') { inputProps.options = props.selectOptions; - inputProps.className = props.value === '' ? 'text-muted border-gray-600' : null; + inputProps.className = props.value === '' ? 'text-muted' : null; } if (props.type === 'checkbox') { inputProps.checked = props.isChecked; @@ -115,7 +114,6 @@ AuthnCustomValidationFormGroup.defaultProps = { invalid: false, ariaInvalid: false, invalidMessage: '', - inputFieldStyle: '', helpText: '', className: '', onClick: null, @@ -140,7 +138,6 @@ AuthnCustomValidationFormGroup.propTypes = { invalidMessage: PropTypes.string, helpText: PropTypes.string, className: PropTypes.string, - inputFieldStyle: PropTypes.string, isChecked: PropTypes.bool, optionalFieldCheckbox: PropTypes.bool, onClick: PropTypes.func, diff --git a/src/common-components/tests/AuthnValidationFormGroup.test.jsx b/src/common-components/tests/AuthnValidationFormGroup.test.jsx index 88f3dfbf..0c765e54 100644 --- a/src/common-components/tests/AuthnValidationFormGroup.test.jsx +++ b/src/common-components/tests/AuthnValidationFormGroup.test.jsx @@ -15,14 +15,14 @@ describe('AuthnCustomValidationFormGroup', () => { it('should show label in place of placeholder when field is empty', () => { const validationFormGroup = mount(); - expect(validationFormGroup.find('label').prop('className')).toEqual('pt-10 focus-out form-label'); + expect(validationFormGroup.find('label').prop('className')).toEqual('pgn__form-label pt-10 focus-out'); }); it('should show label on top of field when field is focused in', () => { const validationFormGroup = mount(); validationFormGroup.find('input').simulate('focus'); - expect(validationFormGroup.find('label').prop('className')).toEqual('pt-10 h6 form-label'); + expect(validationFormGroup.find('label').prop('className')).toEqual('pgn__form-label pt-10 h6'); }); it('should keep label hidden for checkbox field', () => { @@ -32,7 +32,7 @@ describe('AuthnCustomValidationFormGroup', () => { optionalFieldCheckbox: true, }; const validationFormGroup = mount(); - expect(validationFormGroup.find('label').prop('className')).toEqual('sr-only form-label'); + expect(validationFormGroup.find('label').prop('className')).toEqual('pgn__form-label sr-only'); }); it('should keep label hidden when input field is not empty', () => { @@ -41,6 +41,6 @@ describe('AuthnCustomValidationFormGroup', () => { value: 'test', }; const validationFormGroup = mount(); - expect(validationFormGroup.find('label').prop('className')).toEqual('sr-only form-label'); + expect(validationFormGroup.find('label').prop('className')).toEqual('pgn__form-label sr-only'); }); }); diff --git a/src/forgot-password/ForgotPasswordPage.jsx b/src/forgot-password/ForgotPasswordPage.jsx index 6e13b100..ae31596a 100644 --- a/src/forgot-password/ForgotPasswordPage.jsx +++ b/src/forgot-password/ForgotPasswordPage.jsx @@ -117,7 +117,6 @@ const ForgotPasswordPage = (props) => { onChange={e => setFieldValue('email', e.target.value)} helpText={intl.formatMessage(messages['forgot.password.email.help.text'], { platformName })} className="mb-0 w-100" - inputFieldStyle="border-gray-600" /> this.setState({ email: e.target.value, isSubmitted: false })} - inputFieldStyle="border-gray-600" /> this.setState({ password: e.target.value, isSubmitted: false })} - inputFieldStyle="border-gray-600" /> diff --git a/src/login/tests/__snapshots__/LoginPage.test.jsx.snap b/src/login/tests/__snapshots__/LoginPage.test.jsx.snap index cd86e364..f19ef4b2 100644 --- a/src/login/tests/__snapshots__/LoginPage.test.jsx.snap +++ b/src/login/tests/__snapshots__/LoginPage.test.jsx.snap @@ -37,15 +37,14 @@ exports[`LoginPage should match TPA provider snapshot 1`] = ` className="form-group" > { value={values.goals} className="mb-20" onChange={(e) => onChangeHandler('goals', e.target.value)} - inputFieldStyle="border-gray-600" /> ); diff --git a/src/register/RegistrationPage.jsx b/src/register/RegistrationPage.jsx index 212deed2..ff08e1cf 100644 --- a/src/register/RegistrationPage.jsx +++ b/src/register/RegistrationPage.jsx @@ -480,7 +480,6 @@ class RegistrationPage extends React.Component { onBlur={(e) => this.handleOnBlur(e)} onChange={(e) => this.handleOnChange(e)} helpText={intl.formatMessage(messages['helptext.name'])} - inputFieldStyle="border-gray-600" /> this.handleOnBlur(e)} onChange={(e) => this.handleOnChange(e)} helpText={intl.formatMessage(messages['helptext.username'])} - inputFieldStyle="border-gray-600" /> this.handleOnBlur(e)} onChange={(e) => this.handleOnChange(e)} helpText={intl.formatMessage(messages['helptext.email'])} - inputFieldStyle="border-gray-600" /> {!currentProvider && ( this.handleOnBlur(e)} onChange={(e) => this.handleOnChange(e)} helpText={intl.formatMessage(messages['helptext.password'])} - inputFieldStyle="border-gray-600" /> )} this.handleOnBlur(e)} onChange={(e) => this.handleOnChange(e)} selectOptions={this.getCountryOptions()} - inputFieldStyle="border-gray-600" />