Skip to content

Cleanup: fixed some warnings #20344

Open
tejasM17 wants to merge 4 commits intoankidroid:mainfrom
tejasM17:main
Open

Cleanup: fixed some warnings #20344
tejasM17 wants to merge 4 commits intoankidroid:mainfrom
tejasM17:main

Conversation

@tejasM17
Copy link

Description

in kay picker, never used parameter, redundant qualifier and some grammar

Fixes

Approach

address android studio warning in KeyPicker, removed unused parameters,
improve readability, miner grammar in comments, confirmed no changes to dispatchKeyEvent logic

How Has This Been Tested?

build inside studio On-device integration tests
key selection still working, selected key updates ui, no fault observed

Checklist

  • You have a descriptive commit message with a short title (first line, max 50 chars).
  • You have commented your code, particularly in hard-to-understand areas
  • You have performed a self-review of your own code
  • UI changes: include screenshots of all affected screens (in particular showing any new or changed strings)
  • UI Changes: You have tested your change using the Google Accessibility Scanner

@tejasM17 tejasM17 marked this pull request as draft February 20, 2026 10:16
@tejasM17 tejasM17 marked this pull request as ready for review February 20, 2026 10:17
@tejasM17 tejasM17 marked this pull request as draft February 20, 2026 10:18
@tejasM17 tejasM17 marked this pull request as ready for review February 20, 2026 10:25
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.

[Cleanup]: Fix Android Studio Warnings

2 participants

Comments