Skip to content
This repository has been archived by the owner on Aug 13, 2023. It is now read-only.

Add transparent borders on brand component #647

Merged
17 commits merged into from
Jun 20, 2019
Merged

Conversation

ghost
Copy link

@ghost ghost commented Jun 18, 2019

Resolves #646

Overall change: Add transparent borders controlled via props to make the start/end of Brand component clear in high contrast mode.

Code changes:

  • Add optional props borderBottom and borderTop

  • I have assigned myself to this PR and the corresponding issues
  • Tests added for new features
  • Test engineer approval

@ghost ghost added ws-home Tasks for the WS Home Team a11y Accessibility-related task ws-fp-v1 labels Jun 18, 2019
@ghost ghost self-assigned this Jun 18, 2019
@ghost ghost marked this pull request as ready for review June 18, 2019 13:57
@ghost ghost requested review from dr3, j-pendlebury, pjlee11 and sareh as code owners June 18, 2019 13:57
packages/components/psammead-brand/README.md Outdated Show resolved Hide resolved
packages/components/psammead-brand/src/index.jsx Outdated Show resolved Hide resolved
packages/components/psammead-brand/CHANGELOG.md Outdated Show resolved Hide resolved
packages/components/psammead-brand/src/index.jsx Outdated Show resolved Hide resolved
packages/components/psammead-brand/src/index.stories.jsx Outdated Show resolved Hide resolved
sadickisaac and others added 8 commits June 18, 2019 17:37
Copy link
Contributor

@andrew-nowak andrew-nowak left a comment

Choose a reason for hiding this comment

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

cool 👍

Copy link
Contributor

@DenisHdz DenisHdz left a comment

Choose a reason for hiding this comment

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

Nice 👍🏻

@DenisHdz
Copy link
Contributor

This PR has now conflicts.

@WeimingFan
Copy link

Tom and I have checked over and all looks good, thanks

@david-boydell
Copy link

Looks good to me, this is ready for merge.

image

amywalkerdev added a commit that referenced this pull request Jun 20, 2019
Add transparent borders on brand component
@ghost ghost merged commit a78d0bf into latest Jun 20, 2019
@sareh sareh deleted the transparent-brand-border branch June 21, 2019 22:09
This pull request was closed.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
a11y Accessibility-related task ws-home Tasks for the WS Home Team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Brand - Add transparent border for accessibility
7 participants