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

fix: Fixed the font blur issue #172

Merged
merged 1 commit into from
Dec 26, 2024

Conversation

pengfeixx
Copy link
Contributor

Fixed the font blur issue

Bug: https://pms.uniontech.com/bug-view-297093.html
Log: Fixed the font blur issue

Copy link
Contributor

@myk1343 myk1343 left a comment

Choose a reason for hiding this comment

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

+1

Fixed the font blur issue

Bug: https://pms.uniontech.com/bug-view-297093.html
Log: Fixed the font blur issue
@deepin-ci-robot
Copy link

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: myk1343, pengfeixx, rb-union

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

@deepin-ci-robot
Copy link

deepin pr auto review

关键摘要:

  • MainWindow.qml 文件中 windowMiniWidth 属性从 680 更改为 685,确认是否有特定的布局或设计需求需要这一更改。
  • MainWindow.qml 文件中 height 属性从 680 更改为 681,同样需要确认是否有特定的布局或设计需求。
  • MainWindow.qml 文件中 drag.minimumX 属性的更改可能影响拖动逻辑,需要确认新的计算方式是否正确。
  • WebEngineView.qml 文件中新增了一个 Rectangle 元素用于绘制滚动条,但注释掉了 color 属性的设置,需要确认是否应该启用或移除。
  • WebEngineView.qml 文件中移除了一个 Rectangle 元素,但未提供替代方案,需要确认是否有其他方式实现相同的功能。

是否建议立即修改:

  • 确认所有更改是否符合设计需求,并确保没有引入新的布局问题。
  • 检查 drag.minimumX 的更改是否正确,并确保拖动逻辑正常工作。
  • 如果新增的 Rectangle 元素用于滚动条,需要确认 color 属性的设置是否正确,并考虑是否应该启用。
  • 如果移除的 Rectangle 元素是必要的,需要提供替代方案或恢复该元素。

@pengfeixx pengfeixx merged commit 49a586d into linuxdeepin:develop/snipe Dec 26, 2024
17 checks passed
@pengfeixx pengfeixx deleted the bugfix branch December 26, 2024 09:40
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