leavesCZY / Matisse

A Image/Video Selector Implemented with Jetpack Compose(一个用 Jetpack Compose 实现的 Android 图片视频选择框架)
https://github.com/leavesCZY/Matisse/wiki
Apache License 2.0
773 stars 60 forks source link

UI & 功能 更新 #3

Closed FunnySaltyFish closed 2 years ago

FunnySaltyFish commented 2 years ago

此PR包含如下更新:

框架本身仍需加强的:

leavesCZY commented 2 years ago

@FunnySaltyFish 代码貌似有点问题

此外,你为 MatisseActivity 设置 windowNoTitle 的目的是什么呢?

FunnySaltyFish commented 2 years ago

@leavesCZY 感谢Review

  1. 是的,statusBar的隐藏应该在AnimatedVisibility内部,用于在查看图片时隐藏掉状态栏,已修正
  2. 图片缩放在我写了一段时间后发现不那么容易,要实现流畅的效果,得考虑图片的大小与当前控件的大小,根据缩放比例判断拖动的边界。我已将有关拖动的代码删去,目前仅保留双击放大的效果。
  3. Activity设置主题的原因是,在我引入项目时发现,如果整个Application没有设置主题,那么MatisseActivity默认会有个标题栏(也就是应用名字)与页面重叠,因此对Activity本身提前加入了Theme。至于windowNoTitle是否是必须的,我未做额外考量,只是加上了。