Fix intro flow home window frame wrapping#1136
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Repository UI Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (2)
📝 WalkthroughWalkthroughThe changes refactor home page initialization by introducing a helper function that wraps the home page in a VirtualWindowFrame. A new flag prevents duplicate frame application. The navigation flow after intro completion is updated to use this new helper instead of directly instantiating the home page widget. These modifications standardize the windowing behavior across different initialization paths while maintaining existing intro-page rendering logic. 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
Resolve #1132 .
Summary
VirtualWindowFramewrapper for the home page.Testing
lib/app.dart,lib/intro.dart, andlib/view/page/home.dart.Summary by CodeRabbit