FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

fix: 修复安卓触控跳转 & feat. 增加全局歌词偏移设置 by kid141252010 · Pull Request #8 · SPlayer-Dev/SPlayer-for-Android · GitHub

fix: 修复安卓触控跳转 & feat. 增加全局歌词偏移设置 - #8

Merged
Re-BeiChen merged 14 commits into
SPlayer-Dev:devfrom
kid141252010:fix/android-lyric-tap-seek
Apr 26, 2026
Merged

fix: 修复安卓触控跳转 & feat. 增加全局歌词偏移设置#8
Re-BeiChen merged 14 commits into
SPlayer-Dev:devfrom
kid141252010:fix/android-lyric-tap-seek

Conversation

Copy link
Copy Markdown

📌 变更类型

  • ✨ feat — 新功能
  • 🐞 fix — Bug 修复
  • 🎨 style — 仅样式 / 格式,不影响逻辑
  • ♻️ refactor — 重构,不改变外部行为
  • ⚡ perf — 性能优化
  • 📝 docs — 仅文档
  • 🧪 test — 新增 / 修改测试
  • 🔧 chore — 构建 / 脚本 / 依赖
  • 🚀 ci — 仅 CI 配置
  • ⏪ revert — 回滚

📝 变更说明

新增全局歌词偏移设置,并将全局偏移与单曲偏移统一用于歌词显示、歌词索引计算、桌面/悬浮歌词同步等显示链路。

修复安卓端歌词触摸跳转问题:默认歌词和 AMLL 歌词的 seek 目标均只使用原始歌词时间,不再叠加全局偏移、单曲偏移或 AMLL 内部提前时间。AMLL 触摸跳转后,会在命中的歌词行播放期间抵消 AMLL 内部提前量,避免 seek 到该行后歌词动画提前展开,同时保留正常连续播放时的 AMLL 提前入场效果。

🔗 关联 Issue

Closes #

📱 影响范围

  • 🎵 播放引擎 / 音频
  • 📝 歌词 / 桌面歌词
  • 🔔 通知栏 / MediaSession
  • 🌐 在线音乐 (网易云 / Jellyfin / Navidrome / Emby / Subsonic / Last.fm)
  • 🧩 内置 API (nodejs-mobile)
  • 🎨 UI / 主题 / 布局
  • 📦 构建 / 打包 / 签名
  • 📱 Capacitor / 原生 Android 代码
  • 📄 文档 / README

✅ 自检清单

  • 本 PR 目标分支为 master
  • 本地已执行 pnpm lint 且无 warning
  • 本地已执行 pnpm typecheck 且无报错
  • 已在 至少一台真机 上构建并验证关键路径
  • UI 改动已兼顾 手机竖屏 + 平板横屏 两种布局
  • 新增 / 修改的文案使用中文,与项目整体风格一致
  • 未引入不必要的依赖 / 大体积资源
  • 未修改签名密钥 / CI Secrets 相关文件

📸 截图 / 录屏 (UI 类 PR 必填)

改动前 改动后

🧪 测试方式

  1. 安卓真机开启 AMLL / 逐字歌词,触摸未播放歌词行,确认播放进度跳到该行原始时间,动画不提前展开。
  2. 设置全局歌词偏移和单曲歌词偏移后,分别验证 AMLL 歌词和默认歌词触摸 seek 仍跳到原始歌词时间。
  3. 验证 AMLL 歌词区上下滑动不误触 seek,歌词页横向滑动不被歌词触摸误触发整页切页。
  4. 关闭 AMLL 使用默认歌词,确认原触控跳转行为正常。

This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
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


Back | FazBrowse Home | New Git URL