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

Exposed toplevel status as a property for spatial and canvasitem #36092

Closed

Conversation

Duroxxigar
Copy link
Contributor

@Duroxxigar Duroxxigar commented Feb 11, 2020

Addresses #10453

@Duroxxigar Duroxxigar requested a review from a team as a code owner February 11, 2020 00:20
doc/classes/CanvasItem.xml Outdated Show resolved Hide resolved
@willnationsdev
Copy link
Contributor

Since it was last updated in February, I'd recommend rebasing this pull request. Or, if unavailable, mentioning in the related Issue that someone else should handle the PR.

@akien-mga
Copy link
Member

akien-mga commented Sep 30, 2020

We actually discussed this in a live Q&A today, exposing this as a property makes sense. It needs a rebase before it can be merged.

Make sure to run doctool, normally when you have setter, getter and property exposed, the setter and getter should be removed from the documentation (as the property's description is sufficient). See https://docs.godotengine.org/en/latest/community/contributing/updating_the_class_reference.html#updating-the-documentation-template

Might be worth mentioning in the docs how child nodes of a "toplevel" node behave if they are not toplevel themselves (see #24154).

@akien-mga akien-mga added this to the 4.0 milestone Sep 30, 2020
@akien-mga akien-mga added the cherrypick:3.x Considered for cherry-picking into a future 3.x release label Sep 30, 2020
Calinou and others added 21 commits October 1, 2020 02:17
Implement the abstraction by targeting the current VM.
`Godot` workflow is long removed in `master`, and is only found in the `3.2-android-app-bundle` branch currently. As of now Github actions has no way to show `passing` if all checks are passed but only to show the badge of each workflow individually. We can make a feature request later on but for now, we should remove the `Godot CI` badge as it reflects completely something else.
Instead of just the one argument that is exactly --test.
The long-form arguments --test-case and --test-suite were ignored.
This is used to get a direct pointer for the data inside a Variant.
Added as another class instead of in Variant directly because this
should only be used in special cases, so use must be explicit.
-No new features yet
-Unlike godot 3.x, sorting happens using GPU
Added request_raw to HttpRequest
Added decompress_dynamic to Compression class
Added decompress_dynamic to BytePoolArray

Merge doc fix

revert
updated variables to use size_t

removed line break to make code style more consistent

added conditional check to return an error if offset field is used when loading a ZIP package

fixed typo

formatted file

added commit regarding self contained exe files

handled error loging for load zip file with offset

spelling tweak

updated conditional statement for magic check

udpated error message when load Zip file with offset is called

fix CI

Trying to fix CI

fix CI done

Added error message for loading self-contained exe with offset.

Updated documentation.

Fix indent

final fix indent

Updated documentation.

fix indents

Updated doc based on suggestion

Final fix

fixed format
NathanLovato and others added 21 commits October 1, 2020 02:17
Required for godotengine/godot-docs#4074 to pass, we're removing that
page from the docs as part of the getting started section rewrite.
Keeps track of the order in which items are collected by
_collect_ysort_children, and uses that order to break
ties between items with similar Y positions.
Re: Space spacing being in addition to character spacing see:

 * <https://github.com/godotengine/godot/blob/d7b85fbaa1fc438effe406c9d7f973749eb4e527/scene/resources/dynamic_font.cpp#L858-L859>

Re: Value being able to be negative see example here:

 * <godotengine#38630>

But also note that nodes other than `Label` may not currently render extra space spacing correctly.
Co-authored-by: Marcel Admiraal <madmiraal@users.noreply.github.com>
Co-authored-by: Rémi Verschelde <rverschelde@gmail.com>
Changed the documentation of distance_fade_max_distance and min
to be consistent with the behavior of the code as well as the user
documentation. Also noted the behavior of swapping max_distance and
min_distance (where max_distance < min_distance).

Fixes godotengine#36051.

Co-authored-by: Clay John <claynjohn@gmail.com>
The privacy here is already private
Entering text will now start searching automatically after 0.25 seconds
have passed (debounce delay).

This removes the need for a separate Search button.
@Duroxxigar
Copy link
Contributor Author

uhhh - oops. This is why you don't work on stuff so late in the night I guess. Really don't feel like fixing this PR.

Closing and just created a new one - #42448

@Duroxxigar Duroxxigar closed this Oct 1, 2020
@akien-mga akien-mga added archived and removed cherrypick:3.x Considered for cherry-picking into a future 3.x release labels Oct 1, 2020
@Duroxxigar Duroxxigar deleted the make-spatial-toplevel-property branch May 2, 2021 03:21
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.

None yet