Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Nov 18, 2025

Onboarding pages were using deprecated Paletter class with hard-coded colors and fixed 400px widths, breaking theme support and desktop optimization.

Changes

New Component

  • ResponsiveCenter widget using LayoutBuilder to constrain content width (600-800px) on desktop while maintaining full width on mobile

Theme Integration

Replaced deprecated Paletter methods with theme colors:

// Before
color: Paletter.getExtraDarkGray(context)

// After  
color: Theme.of(context).colorScheme.surfaceContainerLow

Applied Material Design 3 tokens:

  • surfaceContainerLow for backgrounds
  • outline for borders
  • onSurfaceVariant for secondary text
  • inverseSurface for active states

Responsive Layouts

Removed fixed widths in favor of flexible constraints:

// Before
Container(
  width: 400,
  child: longButton(...)
)

// After
ResponsiveCenter(
  maxWidth: 600,
  child: SizedBox(
    width: double.infinity,
    child: longButton(...)
  )
)

Files Modified

  • 1 new: responsive_center.dart
  • 12 updated: all onboarding pages (page01, name, picture, login variants, done, invited_by, profile, starter_pack, follow_graph)

Impact

  • Theme switching now works in onboarding
  • Desktop users get constrained, centered content
  • Mobile layouts unchanged
  • Zero deprecated code in onboarding flow
Original prompt

This section details on the original issue you should resolve

<issue_title>onboarding does not support theme</issue_title>
<issue_description>Right now the onboarding pages do not use the app theme and are not optimized for large desktop screens</issue_description>

<agent_instructions>refactor the code to use the app theme in the onboarding section and use layout builder to optimize for desktop and mobile</agent_instructions>

Comments on the Issue (you are @copilot in this section)


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Copilot AI and others added 3 commits November 18, 2025 15:53
- Replace deprecated Paletter class usage with Theme.of(context)
- Add ResponsiveCenter widget for better desktop optimization
- Update all onboarding pages to use responsive layouts
- Remove hard-coded 400px widths in favor of flexible layouts
- Use proper theme colors (surfaceContainerLow, outline, onSurfaceVariant, etc.)

Co-authored-by: 1-leo <[email protected]>
- Replace remaining Paletter usage in onboarding_done.dart
- Update onboarding_starter_pack.dart with theme colors and responsive layout
- Update onboarding_follow_graph.dart with theme colors
- Replace all hard-coded 400px widths with flexible layouts

Co-authored-by: 1-leo <[email protected]>
- Replace 400px widths with double.infinity in onboarding_invited_by.dart
- Replace 400px width with double.infinity in onboarding_profile.dart
- Replace 400px width with double.infinity in onboarding_follow_graph.dart
- All onboarding pages now use flexible, responsive layouts

Co-authored-by: 1-leo <[email protected]>
Copilot AI changed the title [WIP] Refactor onboarding to use app theme and optimize layout Refactor onboarding to use theme system and responsive layouts Nov 18, 2025
Copilot AI requested a review from 1-leo November 18, 2025 16:02
Copilot finished work on behalf of 1-leo November 18, 2025 16:02
@1-leo 1-leo marked this pull request as ready for review November 19, 2025 16:24
@1-leo
Copy link
Collaborator

1-leo commented Nov 19, 2025

not perfect but usable for now

@1-leo 1-leo requested a review from nogringo November 19, 2025 16:25
Copy link
Collaborator

@nogringo nogringo left a comment

Choose a reason for hiding this comment

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

Image

The checkbox must be different.

@1-leo 1-leo requested a review from nogringo November 20, 2025 10:01
@1-leo 1-leo merged commit 378f739 into dev Nov 20, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

onboarding does not support theme

3 participants