Skip to content

MSFT-22 #2923

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

Merged
merged 3 commits into from
Jul 26, 2019
Merged

MSFT-22 #2923

merged 3 commits into from
Jul 26, 2019

Conversation

r0hit-gupta
Copy link
Collaborator

changes for #2745

@codeMinter
Copy link
Contributor

@r0hit-gupta please use react-ui for changes.

@veshu
Copy link
Contributor

veshu commented Jul 25, 2019

@r0hit-gupta Please fix the earliest.

@r0hit-gupta
Copy link
Collaborator Author

r0hit-gupta commented Jul 25, 2019

@veshu
Copy link
Contributor

veshu commented Jul 25, 2019

@r0hit-gupta Please confirm this if we need this PR is needed or w.r.t. topcoder-archive/topcoder-platform-topcoder-react-ui-kit#19

@asadath1395 Please review this.

@r0hit-gupta
Copy link
Collaborator Author

r0hit-gupta commented Jul 25, 2019

@veshu this PR is required too.

@veshu
Copy link
Contributor

veshu commented Jul 25, 2019

@r0hit-gupta Is the PR ready for review? if so remove the code review failed label.

@@ -28,17 +28,17 @@ $track-radius-4: $corner-radius * 2;

&.design,
&.generic {
background: $tc-light-blue;
background: $tc-light-blue-110;
Copy link
Contributor

Choose a reason for hiding this comment

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

@r0hit-gupta This doesn't pass WCAG level AA

If we use tc-dark-blue-110 it will pass

@@ -55,19 +55,19 @@ $track-radius-4: $corner-radius * 2;

&.design,
&.generic {
color: $tc-light-blue;
color: $tc-light-blue-110;
Copy link
Contributor

Choose a reason for hiding this comment

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

@r0hit-gupta Same here it doesn't meet the ratio

using tc-dark-blue-110 for text meets this.

@r0hit-gupta
Copy link
Collaborator Author

@veshu I have updated the colors. Thanks.

@sushilshinde sushilshinde merged commit 02bec8a into topcoder-platform:qa-accessibility Jul 26, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants