fix: dashboard & inner app pages UI cleanup (closes #118)#155
Closed
prasanna-0806 wants to merge 34 commits into
Closed
fix: dashboard & inner app pages UI cleanup (closes #118)#155prasanna-0806 wants to merge 34 commits into
prasanna-0806 wants to merge 34 commits into
Conversation
- Updated package name from "func-kode" to "site" in package.json and package-lock.json. - Added "posthog-js" dependency to package.json and package-lock.json. - Modified .gitignore to include .env.example. - Updated TypeScript configuration for JSX and added new paths. - Integrated PostHogProvider in Providers component and added tracking for GitHub login attempts in LoginForm component.
… client - Updated `createClient` to `createClientComponentClient` in various components for better integration with Supabase. - Simplified the `OnboardButton` and `OnboardProfileForm` components to enhance user experience during onboarding. - Removed deprecated environment variable references and improved error handling in the onboarding process. - Cleaned up the `CONTRIBUTING.md` file by removing outdated links and instructions. - Adjusted TypeScript configuration to align with Next.js standards.
- Replaced direct PostHog calls with a new analytics tracking system in `SignUpForm` for `SIGNUP_ATTEMPTED` and `SIGNUP_FAILED`. - Updated `PostHogProvider` to remove unnecessary environment checks for analytics. - Added new analytics event constants for sign-up actions in `events.ts`. - Documented new events in the analytics documentation.
- Replaced `createClientComponentClient` with `createClient` across various components for improved Supabase integration. - Added new analytics events for dashboard interactions, including `DASHBOARD_VIEWED` and `DASHBOARD_ACTION_CLICKED`. - Updated `.env.example` to clarify PostHog host options for EU and US regions. - Improved error handling and path sanitization in the authentication callback logic.
Resolve conflicts in dashboard layout (profiles admin check from dev) and .env.example (combine local Supabase vars with PostHog docs).
- Replaced `createMiddlewareClient` with `createServerClient` for improved session handling. - Enhanced cookie management by implementing custom `getAll` and `setAll` methods. - Updated session retrieval to use `getUser` instead of `getSession` for better user context.
- Moved the `getAuthorName` function to a dedicated location for better readability and maintainability. - Updated `OnboardButton` to use a ref for the Supabase client, improving performance. - Enhanced `OnboardProfileForm` with analytics tracking for onboarding events. - Improved PostHog initialization logic in `PostHogProvider` to prevent redundant initializations. - Added cookie management functions in middleware to handle Supabase auth cookies more effectively.
Fix/UI tweaks
Resolve conflicts in auth callback route and package-lock.json.
…ice role key; refactor DashboardLayout to use server Supabase client directly.
- Added error tracking for onboarding failures in OnboardProfileForm. - Updated PostHogProvider to ensure proper user identity synchronization and initialization. - Improved session validation in submit-project page to redirect unauthenticated users. - Documented new analytics events for onboarding failures and errors in analytics.md. - Enhanced middleware to handle Supabase user retrieval errors more effectively.
…e client usage in profile API; enhance error handling in DashboardLayout for profile fetching; add profiles table with RLS policies in schema.sql.
…g-109-110 Update project configuration and dependencies
…e-service-role Fix/supabase service role
Fix/UI tweaks
Staging/demo -> main
fix: sync package-lock.json with package.json (add yaml@2.9.0)
Staging/demo
Untrack backup.sql, db_cluster backup, components.zip, scratch scripts, and unused logo variants accidentally included in previous commit. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
17 tasks
17 tasks
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Part of #116
🎯 Summary
Focused UI cleanup pass across all inner app pages — spacing, typography, dark mode, and mobile layout. No redesigns, no data fetching changes.
✅ Spacing
py-8–py-12for section padding. Removed arbitrarymt-5,px-3usages across/events/[id],/submit-project,/blogp-6. Fixed/blogcards which were usingp-5gap-4/space-y-2consistently across/onboard,/profile,/submit-project✅ Typography
<h1>consistent — all inner app pages usetext-3xl font-boldfor page headings (/dashboard,/profile,/onboard,/blog/[slug])text-xs/text-smnow only used on labels, helper text, and metadata (author, date)text-gray-400,text-gray-500, opacity classes withtext-muted-foregroundtoken across all pages✅ Mobile layout (375px)
/dashboardKPI cards —grid md:grid-cols-2 lg:grid-cols-3collapses to single column cleanly at 375px/submit-projectform — submit button changed tow-full md:w-auto, form inputs full-width, no clipping at 375px/events/[id]detail card — date and location changed from single rigid flex row toflex-wrapwith separate<span>elements. CTAs full-width at mobile/blog/[slug]article —max-w-2xl px-4container readable at 375px. Like button and comment form accessible on narrow screens✅ Dark mode
/onboardform card was solid white in dark mode →dark:bg-zinc-900with dark text throughout/dashboard,/events,/blog/[slug]headings usingtext-brand-blue(hardcoded navy) now havedark:text-white/events"Upcoming Events" count was navy-on-dark (invisible) → addeddark:text-whiteborder-0withborder border-borderon:/events— stat cards, upcoming event cards, past event cards, empty state card, CTA card/dashboard— action cards, GitHub integration card, getting started section/blog— blog list cardsOther changes
/raccoon.pngreferences with/landing/logo.pngsite-wide:/blog,/blog/[slug],/dashboard,/onboard,/footer,/auth/login,/auth/sign-up,/docs,/learn,/not-found📝 Notes for reviewers
Blog image migration — any existing Supabase
blogsrecords with the old raccoon image should be updated:public.profilestable — the dashboard layout guard introduced in #145 queriespublic.profiles(withidandis_onboardedcolumns). This table must exist in Supabase for/dashboardto load correctly.✅ Acceptance criteria
npm run buildpasses with no type errorsCloses #118