Skip to content

Document CompletionInfo input methods - #575

Open
jonathanpeppers wants to merge 2 commits into
mainfrom
jonathanpeppers-document-input-method-apis
Open

Document CompletionInfo input methods#575
jonathanpeppers wants to merge 2 commits into
mainfrom
jonathanpeppers-document-input-method-apis

Conversation

@jonathanpeppers

Copy link
Copy Markdown
Member

Refs #191

Sources

Validation

  • Parsed docs/xml/Android.Views.InputMethods/CompletionInfo.xml as XML.
  • Confirmed CRLF line endings with no bare LF characters.
  • Confirmed the focused file has 2 remaining placeholders, both for the .NET-only JniPeerMembers interop property.
  • Reviewed git diff --check and the one-file diff.

Refs #191

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

This PR improves the generated XML documentation for Android.Views.InputMethods.CompletionInfo, replacing placeholder text with meaningful parameter/value/returns descriptions and adding source-reference links to the corresponding Android platform documentation (per #191).

Changes:

  • Documented constructor parameters (id, index, text, label) and added Android reference links for both constructors.
  • Replaced “To be added.” placeholders for several members (e.g., Creator, DescribeContents, Label, Text) and added missing Android reference links where appropriate.
  • Improved value text for key properties (Id, Index, Label*, Text*) to be more descriptive and consistent.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread docs/xml/Android.Views.InputMethods/CompletionInfo.xml Outdated
Refs #191

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
@jonathanpeppers
jonathanpeppers requested review from dalexsoto and removed request for dalexsoto August 15, 2026 16:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants