Desktop: Open file dialogs in document parent folder#3934
Desktop: Open file dialogs in document parent folder#3934timon-schelling wants to merge 1 commit intomasterfrom
Conversation
Summary of ChangesHello, I'm Gemini Code Assist1! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed! This pull request significantly improves the usability of save and export operations by ensuring that file dialogs open in a more intuitive location. By passing the document's parent folder directly to the frontend, users will find it easier to save and export files to relevant directories without manual navigation, streamlining their workflow. Highlights
Using Gemini Code AssistThe full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips. Invoking Gemini You can request assistance from Gemini at any point by creating a comment using either
Customization To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a Limitations & Feedback Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for GitHub and other Google products, sign up here. Footnotes
|
There was a problem hiding this comment.
Code Review
This pull request correctly implements opening file dialogs in the document's parent folder for "Save As" and export operations, which is a nice ergonomic improvement. I've found a regression in the save logic that could lead to an incorrect saved status for the document. My review includes a suggestion to fix this.
editor/src/messages/portfolio/document/document_message_handler.rs
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
1 issue found across 6 files
Prompt for AI agents (unresolved issues)
Check if these issues are valid — if so, understand the root cause of each and fix them. If appropriate, use sub-agents to investigate and fix each issue separately.
<file name="editor/src/messages/portfolio/document/document_message_handler.rs">
<violation number="1" location="editor/src/messages/portfolio/document/document_message_handler.rs:840">
P1: `SaveDocument` no longer marks the document as saved, so the unsaved indicator can remain stale after a successful save.</violation>
</file>
Reply with feedback, questions, or to request a fix. Tag @cubic-dev-ai to re-run a review.
editor/src/messages/portfolio/document/document_message_handler.rs
Outdated
Show resolved
Hide resolved
7a66494 to
4e90ea8
Compare
Open dialogs in documents parent folder if possible, to make Save As and exports more ergonomic.