-
Notifications
You must be signed in to change notification settings - Fork 0
Add content to Overview tab with SectionCard + List component #8
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
base: main
Are you sure you want to change the base?
Conversation
src/main.tsx
Outdated
| <Routes> | ||
| //TODO: Add more routes that takes you to different template pages | ||
| <Route path="/" element={<App />} /> | ||
| <Route index element={<LibraryPage />} /> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
App needs to be rendered here. In App, we can have different elements being rendered, but main.tsx shouldn't know of our components
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, Done.
src/pages/LiberyPage/index.tsx
Outdated
| import { Typography as T } from '../../core/typography' | ||
| import Tabs from '../../components/common/organisms/Tabs' | ||
| import { Main } from '../../components/common/styled' | ||
| import { LIBRARY_TABS } from './tabs.config' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
why caps 😄
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Changed to LiberyTabs
| @@ -0,0 +1,32 @@ | |||
| export const overviewSections = [ | |||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
why .config?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I used .config here to make it clear this file is only exporting static configuration data (no logic). Happy to rename it. I can rename it to overview.ts to keep it simple.
Pull Request
Description
This PR adds info under the Overview section.
To achieve this, I also introduced some new building blocks to make the structure more scalable:
The Overview tab now contains:
How to test
Install and start the project:
npm installnpm run devOpen the Overview tab and Verify that:
Check styling:
Type of Change
feature: New featurechore: Maintenance, dependency updates, or refactoringtest: Adding or improving testsbug: Bug fixdocs: Documentation updatesShortcut story
https://trello.com/c/Hq1p9oUC/14-feature-layout-tab-preview-overview
Related Issues
Testing Performed
Screenshots/Recordings
Checklist
Additional Notes