Skip to content

fix: folder icons did not update with the theme#293

Merged
zsien merged 1 commit intolinuxdeepin:masterfrom
zsien:zs/fix/foldericon
Jun 4, 2024
Merged

fix: folder icons did not update with the theme#293
zsien merged 1 commit intolinuxdeepin:masterfrom
zsien:zs/fix/foldericon

Conversation

@zsien
Copy link
Contributor

@zsien zsien commented May 28, 2024

@zsien zsien requested a review from a team May 28, 2024 07:47
@zsien zsien force-pushed the zs/fix/foldericon branch 2 times, most recently from 448f57c to 956d3e7 Compare May 29, 2024 02:04
Copy link
Member

@BLumia BLumia left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

有办法通过其它方式通知 folder-icon 图标引擎更新图标吗?主要这个文件夹图标 可能 还会在别的地方复用,如果不走图标引擎获取的话就意味着以后调样式要调多个地方的样式。

@zsien
Copy link
Contributor Author

zsien commented May 29, 2024

有办法通过其它方式通知 folder-icon 图标引擎更新图标吗?主要这个文件夹图标 可能 还会在别的地方复用,如果不走图标引擎获取的话就意味着以后调样式要调多个地方的样式。

url 加上主题名应该可以。但我觉得复用的话,复用 QML 也完全没问题。

@deepin-bot
Copy link

deepin-bot bot commented May 30, 2024

TAG Bot

New tag: 0.7.0
DISTRIBUTION: unstable
Suggest: synchronizing this PR through rebase #297

@BLumia
Copy link
Member

BLumia commented May 30, 2024

cc @kegechen

@kegechen kegechen force-pushed the zs/fix/foldericon branch from 956d3e7 to 6d0fdce Compare June 3, 2024 06:38
@zsien zsien force-pushed the zs/fix/foldericon branch from 6d0fdce to 73a9453 Compare June 3, 2024 09:42
@deepin-ci-robot
Copy link

deepin pr auto review

关键摘要:

  • GridLayout中使用了Repeater来创建图标,但没有考虑到icons数组的长度可能不足4的情况,可能会导致布局异常。
  • DciIcon组件的sourceSize属性被设置为parent.width / 2, parent.height / 2,但没有考虑到parent的尺寸可能不是整数的情况,可能会导致布局异常。
  • color属性被硬编码为"#26FFFFFF",这可能不是一个好的做法,因为它限制了组件的可重用性。
  • radius属性被硬编码为12,同样地,这可能不是一个好的做法,因为它限制了组件的可重用性。
  • GridLayoutrowscolumns属性被硬编码为2,这可能不是最佳的设计选择,应该根据实际需求动态计算。
  • GridLayoutanchors.margins属性被硬编码为8,这可能不是最佳的设计选择,应该根据实际需求动态计算。
  • DciIcon组件的palette属性被硬编码为ApplicationHelper.DarkType,这可能不是最佳的设计选择,应该根据实际需求动态设置。
  • DciIcon组件的theme属性被硬编码为ApplicationHelper.DarkType,这可能不是最佳的设计选择,应该根据实际需求动态设置。

是否建议立即修改:

@deepin-ci-robot
Copy link

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: 18202781743, zsien

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

Details 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

@zsien zsien merged commit ba25d95 into linuxdeepin:master Jun 4, 2024
@zsien zsien deleted the zs/fix/foldericon branch June 4, 2024 02:15
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