Matisse icon indicating copy to clipboard operation
Matisse copied to clipboard

UI & 功能 更新

Open FunnySaltyFish opened this issue 2 years ago • 2 comments

此PR包含如下更新:

  • 设置Activity主题为不带ActionBar样式
  • 异形屏适配(刘海、水滴)
  • 读取图片时添加了Indicator
  • 添加简单的图片预览缩放支持
  • 弹出类别的菜单ui优化
  • 补充了一些contentDescription

框架本身仍需加强的:

  • 图片加载速度,目前太慢了(真机,5000张图片,好几秒)
  • 图片滑动卡顿问题
  • 预览图片的缩放、拖动等优化
  • 选择图片时的卡顿问题

FunnySaltyFish avatar Oct 07 '22 13:10 FunnySaltyFish

@FunnySaltyFish 代码貌似有点问题

  • 进入 MatisseActivity 后 statusbar 就被隐藏了,造成整个页面偏移
  • 图片的缩放功能用起来并不流畅,并未达到完全可用的程度

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

leavesCZY avatar Oct 09 '22 03:10 leavesCZY

@leavesCZY 感谢Review

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

FunnySaltyFish avatar Oct 09 '22 03:10 FunnySaltyFish