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

Feature/20/create-layouts #33

Merged
merged 28 commits into from
Nov 27, 2024
Merged

Feature/20/create-layouts #33

merged 28 commits into from
Nov 27, 2024

Conversation

AmirAgassi
Copy link
Member

@AmirAgassi AmirAgassi commented Nov 19, 2024

📐 Create Layouts #20

🔍 What's Included

  • Restructured layout system into components, templates, and pages
  • Added reusable DashboardTemplate with customizable props
  • Created AdminDashboard and UserDashboard layouts
  • Added admin and user routes with respective dashboard pages

🎯 Base Components

  • Container: Width/background wrapper
  • Stack: Flexbox-based stacking layout
  • Grid: Responsive CSS Grid layout
  • Section: Full-width sections with padding controls
  • PageLayout: Full-screen layout wrapper
  • Header: Top navigation container
  • Footer: Bottom footer container
  • FormContainer: Card-like container for forms

🏗️ Templates

  • DashboardTemplate: Reusable dashboard shell with:
    • Customizable logo, navigation tabs, menu items, and actions
    • Responsive sidebar navigation
    • Consistent header and content areas

📄 Page Layouts

  • AdminDashboard: Admin-specific layout with admin menu and actions
  • UserDashboard: User-specific layout with user menu and actions

🛣️ Routes

  • User routes: /dashboard, /projects, /resources, etc.
  • Admin routes: /admin/dashboard, /admin/projects, /admin/users, etc.

🔄 Changes

  • Removed old DashboardLayout, to use the new template system
  • Updated component exports and routing config stuff
  • Added AdminDashboardPage example
  • Simplified page components by using the new layout templates

@AmirAgassi AmirAgassi self-assigned this Nov 19, 2024
@AmirAgassi AmirAgassi linked an issue Nov 19, 2024 that may be closed by this pull request
3 tasks
@aidantrabs
Copy link
Member

merge conflicts - otherwise lgtm

aidantrabs
aidantrabs previously approved these changes Nov 22, 2024
Copy link
Contributor

@juancwu juancwu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm 👍

@AmirAgassi AmirAgassi merged commit 65ca9de into main Nov 27, 2024
4 checks passed
@AmirAgassi AmirAgassi deleted the 20-fe-create-layouts branch November 27, 2024 18:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: No status
Development

Successfully merging this pull request may close these issues.

FE: Create layouts
3 participants