Document CheckBoxPreference overrides - #573
Conversation
Refs #145 Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
There was a problem hiding this comment.
Pull request overview
This PR updates the API reference XML for Android.Preferences.CheckBoxPreference by replacing placeholder documentation with sourced summaries/values for several override properties, aligning this type’s documentation more closely with the upstream Android reference and the existing TwoStatePreference docs in the repo.
Changes:
- Replaced
To be added.placeholders with concrete<summary>/<value>text forChecked. - Added concrete
<summary>/<value>text forDisableDependentsState. - Added concrete
<summary>/<value>text forSummaryOffFormattedandSummaryOnFormatted.
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
|
@dalexsoto review |
dalexsoto
left a comment
There was a problem hiding this comment.
SetSummaryOff(int) and SetSummaryOn(int) are public overrides but still retain To be added. summaries and parameter docs, leaving the stated override-documentation work incomplete. Please document both resource overloads, including summaryResId, and link them to the formatted summary properties: CheckBoxPreference.xml.
Refs #145
Sources
Validation
[xml](Get-Content -Raw 'docs\\xml\\Android.Preferences\\CheckBoxPreference.xml')git diff --checkDocuments four source-backed
CheckBoxPreferenceoverride properties (eight placeholder fields). Twenty placeholders in this type and 293 across the remaining namespace are intentionally left for later sourced slices.