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

[DOCS] TabContainer class descriptions #12055

Merged
merged 1 commit into from
Oct 13, 2017

Conversation

brunosxs
Copy link
Contributor

No description provided.

</description>
</method>
<method name="get_tab_align" qualifiers="const">
<return type="int" enum="TabContainer.TabAlign">
</return>
<description>
Return tab alignment, from the ALIGN_* enum.
Returns the tab alignment, from the ALIGN_* enum.
Copy link
Contributor

@cbscribe cbscribe Oct 12, 2017

Choose a reason for hiding this comment

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

Change the , to a . and make the last sentence conform to the enum standard: See the ALIGN_* constants. (like you did below in the members description)

</member>
<member name="tabs_visible" type="bool" setter="set_tabs_visible" getter="are_tabs_visible">
If [code]true[/code], all tabs that are children of the TabContainer will be visible.
Copy link
Contributor

Choose a reason for hiding this comment

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

Remove the ,

@brunosxs
Copy link
Contributor Author

Done.

@ghost ghost added this to the 3.0 milestone Oct 13, 2017
@@ -157,16 +160,19 @@
<argument index="0" name="visible" type="bool">
</argument>
<description>
Set whether the tabs should be visible or hidden.
If [code]true[/code] all the tabes will be visible.
Copy link
Contributor

Choose a reason for hiding this comment

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

tabes -> tabs

@brunosxs
Copy link
Contributor Author

Sorry for the typo

@cbscribe
Copy link
Contributor

Thanks for your contribution!

@cbscribe cbscribe merged commit 7009ffa into godotengine:master Oct 13, 2017
@brunosxs
Copy link
Contributor Author

I am the one who has to thank you for the patience. I am learning git as I am doing things.
Hope I can help with c++ code in the futre.
Thanks.

@brunosxs brunosxs deleted the tabcontainer branch October 14, 2017 00:42
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.

2 participants