-
Notifications
You must be signed in to change notification settings - Fork 1k
feat: Implement Dev panel & IDE functionalities #1740
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
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 good to me! Reviewed everything up to 19999e6 in 1 minute and 57 seconds
More details
- Looked at
2279
lines of code in20
files - Skipped
0
files when reviewing. - Skipped posting
8
drafted comments based on config settings.
1. packages/models/src/constants/ipc.ts:52
- Draft comment:
Ensure the new channel VIEW_CODE_IN_ONLOOK follows naming conventions and is consistently used in the IPC integration. - Reason this comment was not posted:
Confidence changes required:33%
<= threshold50%
None
2. packages/models/src/ide/index.ts:6
- Draft comment:
The ONLOOK IDE addition looks good. Confirm that the default IDE remains appropriate and that its integration is error-free across the app. - Reason this comment was not posted:
Confidence changes required:0%
<= threshold50%
None
3. packages/models/src/constants/ipc.ts:52
- Draft comment:
New channel VIEW_CODE_IN_ONLOOK added here looks good. Ensure related docs and usage in the dev panel are updated accordingly. - Reason this comment was not posted:
Confidence changes required:0%
<= threshold50%
None
4. packages/models/src/ide/index.ts:6
- Draft comment:
ONLOOK added in IdeType is consistent with the new IDE functionality. Consider if the DEFAULT_IDE should be updated based on user preference. - Reason this comment was not posted:
Confidence changes required:33%
<= threshold50%
None
5. apps/studio/package.json:30
- Draft comment:
The script key 'pree2e' appears to be a potential typographical mistake. If this is meant to run before e2e tests, consider renaming it to something more conventional (like 'pre:e2e') to avoid confusion. - Reason this comment was not posted:
Comment was not on a location in the diff, so it can't be submitted as a review comment.
6. packages/models/src/constants/ipc.ts:34
- Draft comment:
Typo: The constant 'QUIT_AND_INSTALL' has a string value 'quit-and-update-app'. Consider renaming the value to 'quit-and-install' (or adjusting the constant name) for consistency. - Reason this comment was not posted:
Comment was not on a location in the diff, so it can't be submitted as a review comment.
7. packages/models/src/constants/ipc.ts:65
- Draft comment:
Typo: The constant 'GET_TEMPLATE_NODE_CLASS' has a string value 'get-template-node-classes'. The mismatch in singular vs plural could be confusing. Please verify and adjust for consistency. - Reason this comment was not posted:
Comment was not on a location in the diff, so it can't be submitted as a review comment.
8. packages/models/src/constants/ipc.ts:55
- Draft comment:
Typo: The constant 'CLEAN_CODE_KEYS' is assigned the string 'clean-move-keys'. This appears to be a typographical error. Please check if it should be 'clean-code-keys' instead. - Reason this comment was not posted:
Comment was on unchanged code.
Workflow ID: wflow_1uuyuTfRmxiNtLox
You can customize Ellipsis with 👍 / 👎 feedback, review rules, user-specific overrides, quiet
mode, and more.
Title: Undo in one file loads code from another file Summary: When editing and saving a file in Onlook IDE, performing undo twice causes the file content to be replaced by another file’s content. Steps to Reproduce:
Actual Result: Expected Result: |
Title: "View in code" in brand panel does not work in new IDE Summary: In the new IDE, using "View in code" from the brand panel on a selected color does not trigger any action. Steps to Reproduce:
Actual Result: Expected Result: |
Title: Undo from top bar does not revert code changes in IDE Summary: When editing code (e.g., changing the text of a text element) and clicking the Undo button on the top bar, the changes are not reverted in the IDE. Steps to Reproduce:
Actual Result: Expected Result: |
Title: Editing text on canvas does not update code in IDE Summary: When a user edits text directly on the canvas, the corresponding code in the IDE does not reflect the change. Steps to Reproduce:
Actual Result: Expected Result: |
Title: Double-clicking text opens duplicate code tab Summary: When double-clicking a text element on the canvas, instead of navigating to the existing code tab, a duplicate tab is opened. Steps to Reproduce:
Actual Result: Expected Result: |
Title: Selected IDE tab not retained after switching panels Summary: When switching away from the IDE to another panel (e.g., Chat or Style) and returning, the previously selected file tab is not preserved. Steps to Reproduce:
Actual Result: Expected Result: |
Title: Image file opens as raw code in new IDE Summary: When opening an image file in the new IDE, the content is shown as raw encoded text instead of rendering the actual image. Steps to Reproduce:
Actual Result: Expected Result: |
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 is super cool. Awesome work @iNerdStack! There's just 2 small things:
- There's this slight overscroll I'm not sure how to get rid of
- When there's change from outside the page. For example a file is open but it was changed in an IDE, it doesn't update
* implemented code mirror code * implement file scans & other ide functionalities * fix bun lockfile * clean up * more clean up * Move tab change logic * Clean up * fix undo in one file loads code from another file * fix "View in code" in brand panel does not work in new IDE * persist devtab state and improve "view in onlook" functionalities * fix Long tab names break into multiple lines instead of truncating * Ignore next-prod dir
* implemented code mirror code * implement file scans & other ide functionalities * fix bun lockfile * clean up * more clean up * Move tab change logic * Clean up * fix undo in one file loads code from another file * fix "View in code" in brand panel does not work in new IDE * persist devtab state and improve "view in onlook" functionalities * fix Long tab names break into multiple lines instead of truncating * Ignore next-prod dir
Description
This pull request introduces the new
Onlook
IDE and dev panel and adds functionality for scanning and retrieving project files, as well as the ability to view and edit files.Related Issues
related to #1594
Type of Change
Testing
Screenshots (if applicable)
Additional Notes
Important
Introduces
Onlook
IDE with file scanning, retrieval, and editing capabilities, and updates UI and dependencies for integration.Onlook
IDE toide.ts
and integrates it into the system.files-scan.ts
.index.ts
.VIEW_CODE_IN_ONLOOK
andSHOW_EDITOR_TAB
toMainChannels
inipc.ts
.events/files.ts
andevents/code.ts
.DevTab
and file tree components inDevTab/index.tsx
,FileTreeNode.tsx
, andFileTreeRow.tsx
.package.json
for code editing support.This description was created by
for 19999e6. It will automatically update as commits are pushed.