Skip to content

Commit

Permalink
A11y improvements for IDP service & updated accounts & settings embed…
Browse files Browse the repository at this point in the history
….go's
  • Loading branch information
pascalwengerter committed Jul 16, 2021
1 parent 04e5cfa commit 754a049
Show file tree
Hide file tree
Showing 6 changed files with 184 additions and 148 deletions.
35 changes: 32 additions & 3 deletions accounts/pkg/assets/embed.go

Large diffs are not rendered by default.

7 changes: 7 additions & 0 deletions changelog/unreleased/improve-idp-accessibility
Original file line number Diff line number Diff line change
@@ -0,0 +1,7 @@
Bugfix: Improve IDP Login Accessibility

We have addressed the feedback from the `a11y` audit and improved
the IDP login screen accordingly.

https://github.com/owncloud/web/issues/5376
https://github.com/owncloud/web/issues/5377
251 changes: 111 additions & 140 deletions idp/pkg/assets/embed.go

Large diffs are not rendered by default.

2 changes: 1 addition & 1 deletion idp/ui/src/components/ResponsiveScreen.js
Original file line number Diff line number Diff line change
Expand Up @@ -49,7 +49,7 @@ const ResponsiveScreen = (props) => {
} = props;

const logo = withoutLogo ? null :
<img src={process.env.PUBLIC_URL + '/static/logo.svg'} className="oc-logo" alt="ownCloud"/>;
<img src={process.env.PUBLIC_URL + '/static/logo.svg'} className="oc-logo" alt="ownCloud Logo"/>;

const content = loading ? <Loading/> : (withoutPadding ? children : <DialogContent>{children}</DialogContent>);

Expand Down
2 changes: 1 addition & 1 deletion idp/ui/src/containers/Login/Login.js
Original file line number Diff line number Diff line change
Expand Up @@ -106,7 +106,7 @@ class Login extends React.PureComponent {
id="oc-login-password"
{...extraPropsPassword}
/>
{hasError && <Typography id="oc-login-error-message" variant="subtitle2" color="error"
{hasError && <Typography id="oc-login-error-message" variant="subtitle2" component="label" color="error"
className={classes.message}>{errorMessage}</Typography>}
<div className={classes.wrapper}>
<Button
Expand Down
35 changes: 32 additions & 3 deletions settings/pkg/assets/embed.go

Large diffs are not rendered by default.

0 comments on commit 754a049

Please sign in to comment.