Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

perf: 相册启动时间优化,并解决部分bug #176

Merged
merged 1 commit into from
Sep 1, 2023

Conversation

starhcq
Copy link
Contributor

@starhcq starhcq commented Sep 1, 2023

1.未显示的视图,不加载其内容
2.解决重启后,缩略图大小显示不对的问题
3.解决界面宽度调整后,略图列表右侧有较大留白区域的问题
4.使用Dtk日志接口输出日志信息
5.修复相册启动时,侧边栏图标严重闪烁的问题(启动时不加载年/月视图)
6.调整缩略图缓存尺寸为180*180

Log: 相册启动时间优化,并解决部分bug

  1.未显示的视图,不加载其内容
  2.解决重启后,缩略图大小显示不对的问题
  3.解决界面宽度调整后,略图列表右侧有较大留白区域的问题
  4.使用Dtk日志接口输出日志信息
  5.修复相册启动时,侧边栏图标严重闪烁的问题(启动时不加载年/月视图)
  6.调整缩略图缓存尺寸为180*180

Log: 相册启动时间优化,并解决部分bug
@feeengli
Copy link

feeengli commented Sep 1, 2023

[是否满足兼容性要求] Y
[是否满足commit提交规范] Y
[是否满足编码规范] Y
[Review结论] Pass
[Fail原因] N/A

@deepin-ci-robot
Copy link

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: lzwind, starhcq

The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@starhcq starhcq merged commit 1399209 into linuxdeepin:master Sep 1, 2023
17 checks passed
starhcq added a commit that referenced this pull request Sep 6, 2023
  Update version to.6.0.4
修复:
1.chore: url路径转本地路径整改,统一使用localPath接口处理
 * #163
2.修复高分屏图标显示过大的问题
 * #169
3."从文件导入"和设备列表功能完善
 * #170
4.相册挂载外设时,启动时间优化
 * #171
5.内存优化
 * #173
6.相册启动时间优化,并解决部分bug
 * #176
7.优化缩略图列表代理控件加载速度
 * #177
8.[UI] 整改页面为空的显示问题
 * #178
9.修复存在smb路径时,相册闪退的问题
 * #179
10.修复导入链接图片后,相册二次启动失败的问题
 * #180
11.优化缩略图显示效果
 * #181

Log: Update version to 6.0.4
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.

4 participants