Skip to content

fix(mobile): resolve touch issues and unify button styling in plugins settings - #502

Open
Himanth-reddy wants to merge 1 commit into
ProdigyV21:mainfrom
Himanth-reddy:fix/plugins-mobile-touch-and-ui
Open

fix(mobile): resolve touch issues and unify button styling in plugins settings#502
Himanth-reddy wants to merge 1 commit into
ProdigyV21:mainfrom
Himanth-reddy:fix/plugins-mobile-touch-and-ui

Conversation

@Himanth-reddy

Copy link
Copy Markdown
Collaborator

Summary

This PR fixes touch interaction issues on Android mobile devices within the Plugins settings screen and aligns button and category styling with the rest of the mobile settings sections.

Changes

  1. Mobile Touch Layout in PluginScreen:
    • Added a dedicated mobile layout branch using LocalDeviceType.current.isTouchDevice() so touch events trigger directly without TV D-Pad focus slot overrides.
    • Replaced TV-only androidx.tv.material3.Surface dialog buttons in AddRepoDialog with touch/focus-friendly Compose Box buttons.
  2. Unified Styling via MobileSettingsCategory and MobileSettingsRow:
    • Shared MobileSettingsCategory and MobileSettingsRow in SettingsRows.kt.
    • Used standard mobile category cards (BackgroundElevated, RoundedCornerShape(12.dp)), standard row padding, dividers, value pills, and green toggle switches (SuccessGreen) for plugins settings.

@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.
To continue using code reviews, you can upgrade your account or add credits to your account and enable them for code reviews in your settings.

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.

1 participant