-
Notifications
You must be signed in to change notification settings - Fork 438
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
Controls' properties' types should be given in the documentation. #305
Comments
You can actually help with that, if you want :) |
Thank you! |
Should the changes be like |
(Almost) all properties are actually optional, so let's just mention the main type: If you are hopping on this, please make sure to follow the style used "Value is of type ... [and defaults to ...]" :) |
Bruh, I have a question. Can I just "copy/paste" the values from function (property) definitions and see the def_value from there or is there something more to do, like with the |
Concerning stuffs like |
Got it! As many of the |
Now there are conflicting types, there is a |
Nice suggestion, thanks. So, we could have "Aliases" page under "References/Types" section of the docs. Will you like to create the file for this? (
If we have this aliases page, we could then set the value-type of all padding-related props to This will fix the conflict, right? |
Sure! |
A quick question @ndonkoHenri
I have sent a pull request in exactly this format. |
Can we close this issue as completed @taaaf11 ? |
For now, the laptop on which I write this code is not in working condition, and I don't know how much I would have to wait for that. As this issue is related to all controls, I think it should be kept opened. What is your opinion? |
Alright, no prob, let's keep it opened. |
It would be easier if types of properties are also given in the documentation.
Perhaps like this:
value: str
The text displayed.
(From the
Text
documentation page.)The text was updated successfully, but these errors were encountered: