Skip to content

Columns component#90

Merged
mwarman merged 5 commits intomainfrom
74-side-navigation-layout
Mar 11, 2025
Merged

Columns component#90
mwarman merged 5 commits intomainfrom
74-side-navigation-layout

Conversation

@mwarman
Copy link
Member

@mwarman mwarman commented Mar 11, 2025

📢 Instructions

  • Begin with a DRAFT pull request.
  • Follow italicized instructions to add detail to assist the reviewers.
  • After completing all checklist items, change the pull request to READY.

🔧 Change Summary

Describe the changes included in this pull request. Link to the associated GitHub issue(s).

  • fixes Side navigation layout #74
  • Added the Columns component
  • Updated the Settings and Components pages to use Columns
  • Created stories for the Columns component
  • Created a sub-page of the Components page for the Columns component

📝 Checklist

Pull request authors must complete the following tasks before marking the PR as ready to review.

  • Complete a self-review of changes
  • Unit tests have been created or updated
  • The code is free of [new] lint errors and warnings
  • Update storybook stories as needed
  • Update project documentation as needed, README, JSDoc, etc.

🧪 Steps to Test

Describe the process to test the changes in this pull request.

The Columns component may be viewed in Storybook. In the app...

  1. Sign in.
  2. Go to the Settings page or the Components page.
  3. Note the page divided into columns with the navigation on the left and the content on the right.

🔗 Additional Information

Optionally, provide additional details, screenshots, or URLs that may assist the reviewer.

  • [...]

@mwarman mwarman self-assigned this Mar 11, 2025
@mwarman mwarman linked an issue Mar 11, 2025 that may be closed by this pull request
@mwarman mwarman marked this pull request as ready for review March 11, 2025 10:14
@mwarman mwarman merged commit 026c960 into main Mar 11, 2025
5 checks passed
@mwarman mwarman deleted the 74-side-navigation-layout branch March 11, 2025 10:14
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.

Side navigation layout

1 participant