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: Repointed a few broken links and fixed some styling bugs throughout the examples. #1632

Merged
merged 3 commits into from
Feb 3, 2025

Conversation

chris-cedrone-cengage
Copy link
Collaborator

Issue: # 1610

What I did

  • Fixed broken links and some styling snafus throughout the docs

Checklist

  • changeset has been added
  • Pull request description is descriptive
  • I have made corresponding changes to the documentation
  • New and existing unit tests pass locally with my changes
  • [N/A] I have added tests that prove my fix is effective or that my feature works

How to test

Link 1

Verify link works: Design > Intro > Getting started > Icon

Link 2

Verify Sketch link and accompanying paragraph has been removed under: Design > Icon > Icon Resources

Link 3

Verify link works: Components > Intro > Usage > ThemeContext.Provider

Link 4

Verify link works: Components > Button > Button Group

Link 5

Verify link works: Components > Dropdown > Button Group

Link 6

Verify link works: Components > Form Group > Radio Group

Link 7

Verify link works: Components > Global Styles > ThemeContext.Provider

Link 8

Verify link works: Components > Transition > Motion.div

Transition examples

Verify the block is inside the container: Components > Transition > Nudge

Incorrect Modal

Verify focus returns to the button when closing any Modal: Components > Modal > Any

Style issue 1

Verify tooltip focus doesn't overlap text in: Components > Intro > Styles and Themes > Customization

Style issue 2

Verify keyboard focus doesn't overlap other buttons on Link inverse button: Components > Button > Inverse > Link inverse button

Style issue 3

Verify spacing between tooltip buttons: Components > Tooltip > Position > Tooltips

Copy link

changeset-bot bot commented Jan 24, 2025

🦋 Changeset detected

Latest commit: bab5c8b

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
react-magma-docs Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

Copy link
Contributor

Copy link
Contributor

@silvalaura silvalaura linked an issue Jan 30, 2025 that may be closed by this pull request
Copy link
Contributor

Copy link
Contributor

@chris-cedrone-cengage chris-cedrone-cengage merged commit c41fd90 into dev Feb 3, 2025
2 checks passed
@silvalaura silvalaura deleted the bugfix/docsBugs branch February 3, 2025 15:17
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.

Docs bugs
3 participants