Skip to content

Add left/right tap zones for channel and settings menus#5

Merged
yunnysunny merged 3 commits into
mainfrom
cursor/add-left-right-menu-listeners-c378
May 2, 2026
Merged

Add left/right tap zones for channel and settings menus#5
yunnysunny merged 3 commits into
mainfrom
cursor/add-left-right-menu-listeners-c378

Conversation

@yunnysunny
Copy link
Copy Markdown
Member

@yunnysunny yunnysunny commented May 2, 2026

Summary

  • 将播放页触摸监听器改为在 ACTION_DOWN 阶段消费事件,确保关闭 PlayerView 控制器时仍能收到后续 ACTION_UP
  • 保留左半屏打开频道列表、右半屏打开设置面板的现有行为。

Testing

  • source ~/.bashrc && export JAVA_HOME=/usr/lib/jvm/java-17-openjdk-amd64 && export ANDROID_HOME=/opt/android-sdk && export ANDROID_SDK_ROOT=/opt/android-sdk && ./gradlew testDebugUnitTest --no-daemon
  • source ~/.bashrc && export JAVA_HOME=/usr/lib/jvm/java-17-openjdk-amd64 && export ANDROID_HOME=/opt/android-sdk && export ANDROID_SDK_ROOT=/opt/android-sdk && ./gradlew assembleDebug --no-daemon

Notes

  • Cursor Cloud 环境没有 Android 模拟器,本次无法直接运行 APK 手动点击验证;已用单测和 Debug APK 构建验证代码链路。
Open in Web Open in Cursor 

cursoragent and others added 2 commits May 2, 2026 07:50
Co-authored-by: 白一梓 <yunnysunny@gmail.com>
Co-authored-by: 白一梓 <yunnysunny@gmail.com>
@cursor cursor Bot changed the title Add left/right shortcuts for channel and settings menus Add left/right tap zones for channel and settings menus May 2, 2026
@yunnysunny yunnysunny marked this pull request as ready for review May 2, 2026 10:01
Copy link
Copy Markdown

@cursor cursor Bot left a comment

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit 05a50ce. Configure here.

Comment thread app/src/main/java/com/whyun/witv/ui/PlayerActivity.java
Co-authored-by: 白一梓 <yunnysunny@gmail.com>
@yunnysunny yunnysunny merged commit e0ef1bd into main May 2, 2026
2 checks passed
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