-
-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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: update Ti.Android.R and Ti.App.Android.R docs #11492
Conversation
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Left a few notes on some phrasing. I would also like keep the type defined as Titanium.Android.R
instead of renaming it to RProxy
. See my comment in the the related PR on the docs-devkit how we could handle that.
@janvennemann Thanks for review.
First problem could be solved in the generator solely. But the second couldn't, we need to change the docs. |
@drauggres We should be able to have
|
Then we'll have |
@drauggres No not really. As you said it is solely used internally to provide typing information. There is no way to create this type so users only access it through the properties Type:
And then the property definitions state: Titanium.Android.R
Titanium.App.Android.R
|
|
In docs before this PR:
R
described as both:R
property ofTi.Android
and classR
(inTi.Android
namespase) that extendsTi.Proxy
R
inTi.App.Android
Ti.App.Android.R
was almost empty.This PR should fix these problems.
Required for correct fix for tidev/docs-devkit#29.