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

refactor(PPDSC-2606): update feature card component & copy #515

Merged
merged 18 commits into from
Dec 20, 2022

Conversation

JohnTParsons
Copy link
Collaborator

PPDSC-2606

What

  1. Background - why this is needed
    Update the feature card component, and copy updated on the relevant Theme, and Patterns section pages on the site, as requested by design

  2. What did you do
    Updated the copy and refactored FeatureCard to use StandaloneLink

  3. What does the reviewers should expect
    Should match Figma specs

I have done:

  • Written unit tests against changes
  • Written functional tests against the component and/or NewsKit site
  • Updated relevant documentation

I have tested manually:

  • The feature's functionality is working as expected on Chrome, Firefox, Safari and Edge
  • The screen reader reads and flows through the elements as expected.
  • There are no new errors in the browser console coming from this PR.
  • When visual test is not added, it renders correctly on different browsers and mobile viewports (Safari, Firefox, small mobile viewport, tablet)
  • The Playground feature is working as expected

Before:

Used to display a button with an arrow in it

After:

Now displays a link followed by an arrow
PPDSC-2606-after

Who should review this PR:

How to test:

To test go to the Docs home page, where two feature cards can be viewed.

@JohnTParsons JohnTParsons requested a review from a team as a code owner December 8, 2022 17:59
@JohnTParsons JohnTParsons force-pushed the refactor/PPDSC-2606-refactor-feature-card branch from 9a2f71d to 4729a89 Compare December 12, 2022 11:51
evgenitsn and others added 11 commits December 13, 2022 20:02
…#512)

* docs(PPDSC-2600): update Assistive Text component storybook scenarios

* docs(PPDSC-2600): update section label

* docs(PPDSC-2600): update scenarios columns

* docs(PPDSC-2600): update layout props
Bumps [loader-utils](https://github.com/webpack/loader-utils) from 1.4.0 to 1.4.2.
- [Release notes](https://github.com/webpack/loader-utils/releases)
- [Changelog](https://github.com/webpack/loader-utils/blob/v1.4.2/CHANGELOG.md)
- [Commits](webpack/loader-utils@v1.4.0...v1.4.2)

---
updated-dependencies:
- dependency-name: loader-utils
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <support@github.com>

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: baburay23 <54642337+baburay23@users.noreply.github.com>
Bumps [qs](https://github.com/ljharb/qs) from 6.5.2 to 6.5.3.
- [Release notes](https://github.com/ljharb/qs/releases)
- [Changelog](https://github.com/ljharb/qs/blob/main/CHANGELOG.md)
- [Commits](ljharb/qs@v6.5.2...v6.5.3)

---
updated-dependencies:
- dependency-name: qs
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <support@github.com>

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: baburay23 <54642337+baburay23@users.noreply.github.com>
Bumps [express](https://github.com/expressjs/express) from 4.17.1 to 4.18.2.
- [Release notes](https://github.com/expressjs/express/releases)
- [Changelog](https://github.com/expressjs/express/blob/master/History.md)
- [Commits](expressjs/express@4.17.1...4.18.2)

---
updated-dependencies:
- dependency-name: express
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <support@github.com>

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: baburay23 <54642337+baburay23@users.noreply.github.com>
src/link/style-presets.ts Outdated Show resolved Hide resolved
site/components/feature-card/feature-card.tsx Outdated Show resolved Hide resolved
site/components/feature-card/feature-card.tsx Show resolved Hide resolved
@JohnTParsons JohnTParsons merged commit a414237 into main Dec 20, 2022
@JohnTParsons JohnTParsons deleted the refactor/PPDSC-2606-refactor-feature-card branch December 20, 2022 09:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants