-
Notifications
You must be signed in to change notification settings - Fork 1
WIP: Miro based UI updates #27
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
…ntime env url for internal docker communication
| op.create_table('chat', | ||
| sa.Column('message', sqlmodel.sql.sqltypes.AutoString(length=1024), nullable=True), | ||
| sa.Column('is_system', sa.Boolean(), nullable=False), | ||
| sa.Column('created_at', sa.DateTime(timezone=True), server_default=sa.text('now()'), nullable=False), | ||
| sa.Column('updated_at', sa.DateTime(), nullable=False), | ||
| sa.Column('id', sa.Uuid(), nullable=False), | ||
| sa.Column('course_id', sa.Uuid(), nullable=False), | ||
| sa.ForeignKeyConstraint(['course_id'], ['course.id'], ondelete='CASCADE'), | ||
| sa.PrimaryKeyConstraint('id') | ||
| ) |
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 could revert this, but I needed it to get the fresh migrations to run on my end without issues with duplicate create chat table.
|
@Alameen688 see this pr too, so that effort is not duplicated #33 |
michaelgichia
left a comment
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.
Reposting for visibility:
I really like your changes, but maybe we could add more as @ayodeji.moronkeji has mentioned.
- Futuristic landing page
- Mobile responsiveness
- Style the components inside components/ui so that the changes are applied throughout the page.
- Share screenshots of dark/light mode.
- Don't remove the theme provider
Overview
Implemented design system based on Miro screenshot across authentication pages and dashboard pages
Key Changes
Sections affected
sb-*system)Layout and colors
Three-column layout: Left sidebar, main content, right sidebar (Project Settings)
Fixed elements: Header and right sidebar now sticky/fixed
Tab navigation: Consistent styling across course pages and quiz sessions
Color scheme:
#101827(dark background)#182130(content background)#1f2937(surface background)AppSidebar: Dynamic course list, fixed "New Project" button
ProjectSettings: Empty state with gear icon, document upload
Quiz Session: Integrated tab navigation matching course page layout
User Account: Moved to top header with dropdown menu
Theme management: Removed light/dark toggle, enforced single dark theme per Miro UI
Layout consistency: Fixed positioning for headers, sidebars, and tabs
Component reuse: Leveraged existing components for new layouts
Screenshots
Authentication Pages
Dashboard & Course Management