Skip to content
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

Expose ImportDock and its children #47985

Closed
wants to merge 1 commit into from

Conversation

trollodel
Copy link
Contributor

Part of godotengine/godot-proposals#2537
Indipendent from #47520

Expose ImportDock, including a bunch of method for interacting with it.

Considerations for docs are the same of #47767.

@trollodel trollodel requested review from a team as code owners April 17, 2021 19:14
@Calinou Calinou added this to the 4.0 milestone Apr 17, 2021
@trollodel trollodel force-pushed the hackable_importdock branch 2 times, most recently from 452dc6e to 0387e38 Compare May 25, 2021 19:37
Comment on lines +186 to +189
if (p_paths.is_empty()) {
return;
}
Copy link
Member

Choose a reason for hiding this comment

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

Should probably emit edited_paths_changed no? Same with single edit.

@KoBeWi
Copy link
Member

KoBeWi commented Aug 18, 2021

Code seems fine. Needs rebase, so it can be tested etc. Also like in #47767, I don't see a way to actually get the import dock.

@trollodel trollodel force-pushed the hackable_importdock branch from 0387e38 to 931b255 Compare August 23, 2021 20:39
@YuriSizov YuriSizov requested a review from a team August 24, 2021 20:03
@trollodel trollodel force-pushed the hackable_importdock branch from 931b255 to e66a805 Compare August 25, 2021 19:23
@trollodel
Copy link
Contributor Author

Closing per discussion in the Godot contributor chat.
Part of this PR will be extracted to make a cleanup PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants