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

style: update main sidebar #89 #107

Merged
merged 1 commit into from
Sep 28, 2023
Merged

style: update main sidebar #89 #107

merged 1 commit into from
Sep 28, 2023

Conversation

srizvi
Copy link
Contributor

@srizvi srizvi commented Sep 28, 2023

  • fix(course-sidebar.tsx): remove import of ModeToggle component
  • feat(course-sidebar.tsx): add import of SidebarQuickLinks component
  • fix(sidebar.tsx): remove import of ModeToggle component
  • feat(sidebar.tsx): add import of SidebarQuickLinks component
  • fix(course-card.tsx): update styling of price element
  • feat(sidebar-quick-links.tsx): create SidebarQuickLinks component

Closes #89
Closes #90

- fix(course-sidebar.tsx): remove import of ModeToggle component
- feat(course-sidebar.tsx): add import of SidebarQuickLinks component
- fix(sidebar.tsx): remove import of ModeToggle component
- feat(sidebar.tsx): add import of SidebarQuickLinks component
- fix(course-card.tsx): update styling of price element
- feat(sidebar-quick-links.tsx): create SidebarQuickLinks component

Closes #89
Closes #90
@srizvi srizvi added style Changes that do not affect the meaning of the code. lms For issues related to the LMS. labels Sep 28, 2023
@srizvi srizvi added this to the Cycle 2 milestone Sep 28, 2023
@srizvi srizvi self-assigned this Sep 28, 2023
@srizvi srizvi merged commit 652fe39 into main Sep 28, 2023
@srizvi srizvi deleted the 89-style-update-main-sidebar branch September 28, 2023 12:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
lms For issues related to the LMS. style Changes that do not affect the meaning of the code.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

style: update course sidebar style: update main sidebar
1 participant