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

Inline link color should be ebon #761

Merged
merged 6 commits into from
Jul 3, 2019
Merged

Inline link color should be ebon #761

merged 6 commits into from
Jul 3, 2019

Conversation

pjlee11
Copy link
Contributor

@pjlee11 pjlee11 commented Jul 3, 2019

Resolves N/A

UX spotted that the default link color should be ebon not postbox. Follows up #707

Overall change: Changes the default inline link text color to be ebon

Code changes:

Changes the default inline link text color to be ebon

Testing notes
Checkout branch, run storybook, in an incognito window check that the inline link story has text color of Ebon #222222


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

@pjlee11 pjlee11 added ws-articles Tasks for the WS Articles Team articles-av-epic labels Jul 3, 2019
@pjlee11 pjlee11 self-assigned this Jul 3, 2019
Copy link
Contributor

@12 12 left a comment

Choose a reason for hiding this comment

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

@pjlee11
Copy link
Contributor Author

pjlee11 commented Jul 3, 2019

Currently live
image

Wrong styling on latest
image

Right styling with this PR
image

@jamesbrumpton
Copy link
Contributor

LGTM

@sareh sareh merged commit 6a3ed59 into latest Jul 3, 2019
@sareh sareh deleted the inline-link-color-ebon branch July 3, 2019 12:35
@jroebu14 jroebu14 mentioned this pull request Jul 10, 2019
3 tasks
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
articles-av-epic ws-articles Tasks for the WS Articles Team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants