-
Notifications
You must be signed in to change notification settings - Fork 10
Workspace Docs #1
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
Conversation
|
|
||
| # Quickstart | ||
|
|
||
| ## Start With a Prompt (4 mins) |
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.
you can review this but the other sections I will add with https://linear.app/flutterflow/issue/DEVR-1080/quickstart later. whatever's done, let's review that.
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.
Pull Request Overview
This PR introduces comprehensive documentation for Dreamflow's workspace interface, establishing a new documentation structure focused on the integrated development environment. The documentation covers the core workspace panels that provide Flutter development tools including widget management, property editing, code editing, and AI-powered assistance.
- Restructures the documentation sidebar to include dedicated "Getting Started" and "Workspace" sections
- Adds detailed documentation for five core workspace panels (modules, content, properties, agent)
- Updates styling and branding elements to align with the new documentation structure
Reviewed Changes
Copilot reviewed 16 out of 24 changed files in this pull request and generated 16 comments.
Show a summary per file
| File | Description |
|---|---|
| src/css/custom.css | Updates typography, colors, and removes redundant dark mode styles |
| sidebars.ts | Restructures navigation with new categorized sections for getting started and workspace |
| firebase.json | Removes extensive redirect rules, simplifying the configuration |
| docusaurus.config.ts | Updates GitHub repository references and removes blog link |
| docs/workspace/ | Adds comprehensive workspace panel documentation with interactive demos |
| docs/get-started/ | Adds welcome and quickstart guides for new users |
| docs/index.md | Comments out support resources section |
Files not reviewed (1)
- .idea/modules.xml: Language not supported
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
src/css/custom.css
Outdated
| color: var(--ifm-text-secondary-color) !important; | ||
| font-size: 14px !important; /* Set body text size */ |
Copilot
AI
Sep 15, 2025
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.
[nitpick] Consider moving the font-size value to a CSS custom property for better maintainability and consistency across the theme system.
|
|
||
| h1 { | ||
| font-size: 2.5rem !important; /* Adjust this value as needed */ | ||
| font-size: 2.2rem !important; /* Adjust this value as needed */ |
Copilot
AI
Sep 15, 2025
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.
[nitpick] The comments 'Adjust this value as needed' are not helpful. Consider using CSS custom properties for these typography scales to maintain consistency and make adjustments easier.
| } | ||
|
|
||
| h2 { | ||
| font-size: 1.8rem !important; /* Adjust this value as needed */ |
Copilot
AI
Sep 15, 2025
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.
[nitpick] The comments 'Adjust this value as needed' are not helpful. Consider using CSS custom properties for these typography scales to maintain consistency and make adjustments easier.
| } | ||
|
|
||
| h3 { | ||
| font-size: 1.4rem !important; /* Adjust this value as needed */ |
Copilot
AI
Sep 15, 2025
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.
[nitpick] The comments 'Adjust this value as needed' are not helpful. Consider using CSS custom properties for these typography scales to maintain consistency and make adjustments easier.
docs/workspace/agent-panel.md
Outdated
| height: '100%', | ||
| colorScheme: 'light' | ||
| }} | ||
| frameborder="0" |
Copilot
AI
Sep 15, 2025
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.
The 'frameborder' attributes should be 'frameBorder' with camelCase capitalization for React/JSX compatibility.
docs/workspace/agent-panel.md
Outdated
| height: '100%', | ||
| colorScheme: 'light' | ||
| }} | ||
| frameborder="0" |
Copilot
AI
Sep 15, 2025
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.
The 'frameborder' attributes should be 'frameBorder' with camelCase capitalization for React/JSX compatibility.
| frameborder="0" | |
| frameBorder="0" |
docs/workspace/agent-panel.md
Outdated
| height: '100%', | ||
| colorScheme: 'light' | ||
| }} | ||
| frameborder="0" |
Copilot
AI
Sep 15, 2025
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.
The 'frameborder' attributes should be 'frameBorder' with camelCase capitalization for React/JSX compatibility.
docs/workspace/agent-panel.md
Outdated
| height: '100%', | ||
| colorScheme: 'light' | ||
| }} | ||
| frameborder="0" |
Copilot
AI
Sep 15, 2025
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.
The 'frameborder' attributes should be 'frameBorder' with camelCase capitalization for React/JSX compatibility.
| frameborder="0" | |
| frameBorder="0" |
| height: '100%', | ||
| colorScheme: 'light' | ||
| }} | ||
| frameBorder="0" |
Copilot
AI
Sep 15, 2025
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.
This file correctly uses 'frameBorder' with camelCase, which is the proper React/JSX syntax. The other files should follow this pattern.
Co-authored-by: Copilot <[email protected]>
…rkspace # Conflicts: # sidebars.ts
pinkeshmars
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.
@PoojaB26 added some comments. Also, I feel the fonts are too small, users on large screens will be sitting farther from the monitor and may find it difficult to read.
| The Agent will create all the necessary screens, components, and functionality based on your description, giving you a fully functional app that you can further customize and deploy. | ||
|
|
||
|
|
||
| <div style={{ |
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 think it would be more impactful to demo an app that’s widely relatable — the kind of apps used by thousands, if not millions, such as a Fitness Tracker, Expense Tracker, Super App, Stock Market app, Language Learning app, Hotel Finder app, Dating app, or Last-Minute Delivery app. Showing that these can be generated with just a simple prompt (instead of needing a large team) would really highlight the true power of Dreamflow.
Also, in the last slide of the arcade, there are overflow errors, which might leave a slightly low impression.
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.
Updated with a music streaming app instead!
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.
Looks really great now! thanks!
|
|
||
| # Quickstart | ||
|
|
||
| ## Start With a Prompt (4 mins) |
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.
What is (4 mins) here? time to read this page? Or time to generate the app?
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.
Time to generate an app, i'll see if i will keep it or not. idea is to show other methods being longer.
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.
ok, no worries. I was thinking that it is not always 4 minutes because mostly it depends on app complexity.
Co-authored-by: pinkeshmars <[email protected]> Co-authored-by: Copilot <[email protected]>
Description
Provide a brief overview of what this documentation update is about. Explain what sections or topics are being added or revised.
Linear ticket and magic word Fixes DEVR-1088, Fixes DEVR-1086, Fixes DEVR-1084, Fixes DEVR-1087, Fixes DEVR-1083, Fixes DEVR-1082
Type of change