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

docs(row and column): add a link to grid component in deprecation warning #3906

Merged
merged 2 commits into from
Apr 13, 2021

Conversation

DipperTheDan
Copy link
Contributor

@DipperTheDan DipperTheDan commented Apr 13, 2021

Proposed behaviour

Adds a link to the Grid component in the deprecation warning for Row and Column.

Current behaviour

No link to component provided. User will have to manually search for the Grid component.

Checklist

  • Commits follow our style guide
    - [ ] Screenshots are included in the PR if useful
  • All themes are supported if required
    - [ ] Unit tests added or updated if required
    - [ ] Cypress automation tests added or updated if required
  • Storybook added or updated if required
    - [ ] Typescript d.ts file added or updated if required
  • Carbon implementation and Design System documentation are congruent

Additional context

Testing instructions

Click on the word Grid in the deprecation warning in the Row MDX example. You should be taken to the Grid documentation.
Screenshot 2021-04-13 at 14 13 37

@codesandbox-ci
Copy link

codesandbox-ci bot commented Apr 13, 2021

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 0dfce65:

Sandbox Source
carbon-quickstart Configuration

@samtjo samtjo marked this pull request as ready for review April 13, 2021 13:18
@samtjo samtjo requested a review from a team as a code owner April 13, 2021 13:18
@DipperTheDan DipperTheDan merged commit 5f63f88 into master Apr 13, 2021
@DipperTheDan DipperTheDan deleted the link-to-grid-component branch April 13, 2021 13:49
@carbonci
Copy link
Collaborator

🎉 This PR is included in version 68.18.0 🎉

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.

5 participants