Skip to content

Conversation

@yasz1234
Copy link

@yasz1234 yasz1234 commented Dec 6, 2025

1.优化推荐页返回顶部体验:实现了右下角按钮的智能显隐逻辑,仅在滑动超过一定阈值后淡入显示,提升页面纯净度。

2.完善筛选菜单交互:推荐页左上角的番剧类型选择箭头现在会随菜单展开/收起进行旋转动画,且下拉列表精准锚定于箭头下方,不再固定位置。

3.增强选中状态反馈:推荐页的下拉菜单现支持高亮显示当前选中的番剧类型标签,提升用户操作辨识度。

4.升级评分图表动效:桌面端番剧详情页的“评分透视”柱状图在鼠标悬停切换时新增了平滑过渡动画。

5.优化数据展示UI:重构了评分数据(百分比+人数)的视觉呈现,并为容器尺寸变化添加了流畅的拉伸动画效果。

1.优化推荐页返回顶部体验:实现了右下角按钮的智能显隐逻辑,仅在滑动超过一定阈值后淡入显示,提升页面纯净度。

2.完善筛选菜单交互:推荐页左上角的番剧类型选择箭头现在会随菜单展开/收起进行旋转动画,且下拉列表精准锚定于箭头下方,不再固定位置。

3.增强选中状态反馈:推荐页的下拉菜单现支持高亮显示当前选中的番剧类型标签,提升用户操作辨识度。
1.升级评分图表动效:桌面端番剧详情页的“评分透视”柱状图在鼠标悬停切换时新增了平滑过渡动画。

2.优化数据展示UI:重构了评分数据(百分比+人数)的视觉呈现,并为容器尺寸变化添加了流畅的拉伸动画效果。
@yasz1234
Copy link
Author

yasz1234 commented Dec 6, 2025

屏幕截图 2025-12-06 164304 image image

@Predidit
Copy link
Owner

Predidit commented Dec 6, 2025

看上去相当不错的PR,我还没来得及进行测试

不过关于 右下角按钮的智能显隐逻辑 ,这部分修改你在移动设备上做过测试吗,按照一般的经验,列表滑动过程中修改组件树会让滑动立即停止,具体表现为进行一次向下滑动时,整个列表会被卡在显示按钮的 breakpoint 处,直到进行第二次滑动

@yasz1234
Copy link
Author

yasz1234 commented Dec 6, 2025

我在我的安卓手机上进行了测试,没有出现上述问题

不过出现了一个新的问题,会导致下拉菜单无法正常关闭

已修复

@Predidit
Copy link
Owner

Predidit commented Dec 6, 2025

这个修复的实现很脏,使用一个 _ignoreNextTap 布尔变量

使用其他方法修复,这里的问题可能是点击事件被多次触发了

@yasz1234
Copy link
Author

yasz1234 commented Dec 6, 2025

好的

目前这个新方案是提升 MenuAnchor 层级并调整点击响应逻辑

@Predidit
Copy link
Owner

Predidit commented Dec 6, 2025

非常感谢你的贡献,但是在仔细地查看了这个实现之后,我暂时不能合并这个PR,因为存在以下问题

  1. 下拉菜单的实现基于 menuanchor ,这个组件本身缺乏动画,并且 flutter 仓库有一个和 menuanchor 动画相关的PR将会很快落地,这会改变 menuanchor 的行为,让我们的定制实现出现问题 [material/menu_anchor.dart] Add animations to MenuAnchor. flutter/flutter#176494

  2. 为了实现评分透视动画,我们实现了一个相当复杂的自定义动画实现,包括引入了 Timer 和额外的状态变量,以及手动计算动画位置,逻辑和可维护性非常脆弱,这些本来应该是 fl_chart 库的内部细节。这里的收益和代价明显不成正比。

非常高兴你对这个项目感兴趣,如果您有意愿继续参与的话,可以从小的PR开始,尽量一个PR只涉及一个功能,这样修改和交流都会方便非常多,例如我们可以从那个 优化推荐页返回顶部体验 的功能开始。

@yasz1234
Copy link
Author

yasz1234 commented Dec 6, 2025

好的,我知道了,十分感谢您的 review

这个PR我关了,期望后续能有机会再次参与到这个项目之中!

1 similar comment
@yasz1234
Copy link
Author

yasz1234 commented Dec 6, 2025

好的,我知道了,十分感谢您的 review

这个PR我关了,期望后续能有机会再次参与到这个项目之中!

@yasz1234 yasz1234 closed this Dec 6, 2025
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