Skip to content
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

fix(tabs): update background colour to improve contrast for selected Tab in alternate variant FE-5120 #5032

Merged
merged 3 commits into from
Apr 12, 2022

Conversation

edleeks87
Copy link
Contributor

fix #5030

Proposed behaviour

Updates the background colour for selected or hovered Tab when Tabs are rendered inside the
Drawer sidebar or when variant is set to alternate

Current behaviour

Contrast is too low when Tabs are rendered in Drawer sidebar or variant is "alternate"

Checklist

  • Commits follow our style guide
  • Related issues linked in commit messages if required
  • Screenshots are included in the PR if useful
  • Unit tests added or updated if required

QA

  • Tested in CodeSandbox/storybook
  • Add new Cypress test coverage if required
  • Carbon implementation matches Design System/designs
  • UI Tests GitHub check reviewed if required

Additional context

Proposed look after updates:

image

Testing instructions

Test tabs--with-alternate-styling story
Test drawer--with-tab-controls story

The following CodeSandbox is an example of the broken behaviour.
You can see the new behaviour by looking at the version in the comment by codesandbox[bot].

…Tab in alternate variant

Updates the background colour for selected or hovered `Tab` when `Tabs` are rendered inside the
`Drawer` `sidebar` or when `variant` is set to `alternate`

fix #5030
@codesandbox-ci
Copy link

codesandbox-ci bot commented Apr 8, 2022

This pull request is automatically built and testable in CodeSandbox.

To see build info of the built libraries, click here or the icon next to each commit SHA.

Latest deployment of this branch, based on commit d733de0:

Sandbox Source
carbon-quickstart Configuration
carbon-quickstart-typescript Configuration
carbon-quickstart PR

@cypress
Copy link

cypress bot commented Apr 8, 2022



Test summary

1521 0 2 0Flakiness 0


Run details

Project carbon
Status Passed
Commit d733de0
Started Apr 12, 2022 2:05 PM
Ended Apr 12, 2022 2:12 PM
Duration 07:25 💡
OS Linux Debian - 10.9
Browser Chrome 91

View run in Cypress Dashboard ➡️


This comment has been generated by cypress-bot as a result of this project's GitHub integration settings. You can manage this integration in this project's settings in the Cypress Dashboard

@DipperTheDan DipperTheDan marked this pull request as ready for review April 8, 2022 11:14
@DipperTheDan DipperTheDan requested review from a team as code owners April 8, 2022 11:14
@DipperTheDan DipperTheDan requested review from samtjo and Parsium April 8, 2022 11:14
@samtjo samtjo removed their request for review April 8, 2022 11:20
@edleeks87 edleeks87 removed the request for review from Parsium April 8, 2022 13:09
@jamime jamime merged commit 8c4b88f into master Apr 12, 2022
@jamime jamime deleted the FE-5120-update-contrast branch April 12, 2022 14:12
@carbonci
Copy link
Collaborator

🎉 This PR is included in version 106.0.1 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

[Tabs] contrast when in Drawer sidebar or using variant="alternate" is too low
7 participants