Skip to content

Conversation

@deepin-ci-robot
Copy link
Contributor

Synchronize source files from linuxdeepin/dtkdeclarative.

Source-pull-request: linuxdeepin/dtkdeclarative#380

@deepin-ci-robot
Copy link
Contributor Author

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: deepin-ci-robot

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

@deepin-bot
Copy link
Contributor

deepin-bot bot commented Aug 21, 2024

TAG Bot

New tag: 6.0.19
DISTRIBUTION: unstable
Suggest: synchronizing this PR through rebase #148

Synchronize source files from linuxdeepin/dtkdeclarative.

Source-pull-request: linuxdeepin/dtkdeclarative#380
@deepin-ci-robot
Copy link
Contributor Author

deepin pr auto review

关键摘要:

  • fixTopLevelWindowFlags函数中,对于Qt::Window类型的窗口,应该检查是否已经设置了Qt::WindowSystemMenuHint,因为Qt::WindowTitleHint | Qt::WindowSystemMenuHint | Qt::WindowMinimizeButtonHint | Qt::WindowMaximizeButtonHint | Qt::WindowCloseButtonHint是默认情况下设置的,如果已经设置了Qt::WindowSystemMenuHint,那么不需要再次设置。
  • fixTopLevelWindowFlags函数中的switch语句应该使用case语句而不是switch,因为switch语句在C++中不支持default分支,而case语句可以。
  • fixTopLevelWindowFlags函数中的if语句应该使用else if而不是else,以避免逻辑错误。
  • fixTopLevelWindowFlags函数中的if语句应该检查flags是否已经包含了Qt::WindowFullscreenButtonHint,然后再进行移除操作,以避免意外的修改。
  • fixTopLevelWindowFlags函数中的if语句应该添加注释说明为什么需要移除Qt::WindowFullscreenButtonHint,以便其他开发者理解这一操作的目的。
  • fixTopLevelWindowFlags函数中的if语句应该添加注释说明为什么需要根据窗口类型设置不同的标志,以便其他开发者理解这一操作的目的。
  • fixTopLevelWindowFlags函数中的if语句应该添加注释说明为什么需要根据窗口类型设置不同的标志,以便其他开发者理解这一操作的目的。
  • fixTopLevelWindowFlags函数中的if语句应该添加注释说明为什么需要根据窗口类型设置不同的标志,以便其他开发者理解这一操作的目的。
  • fixTopLevelWindowFlags函数中的if语句应该添加注释说明为什么需要根据窗口类型设置不同的标志,以便其他开发者理解这一操作的目的。
  • fixTopLevelWindowFlags函数中的if语句应该添加注释说明为什么需要根据窗口类型设置不同的标志,以便其他开发者理解这一操作的目的。
  • fixTopLevelWindowFlags函数中的if语句应该添加注释说明为什么需要根据窗口类型设置不同的标志,以便其他开发者理解这一操作的目的。
  • fixTopLevelWindowFlags函数中的if语句应该添加注释说明为什么需要根据窗口类型设置不同的标志,以便其他开发者理解这一操作的目的。
  • fixTopLevelWindowFlags函数中的if语句应该添加注释说明为什么需要根据窗口类型设置不同的标志,以便其他开发者理解这一操作的目的。
  • fixTopLevelWindowFlags函数中的if语句应该添加注释说明为什么需要根据窗口类型设置不同的标志,以便其他开发者理解这一操作的目的。
  • fixTopLevelWindowFlags函数中的if语句应该添加注释说明为什么需要根据窗口类型设置不同的标志,以便其他开发者理解这一操作的目的。
  • fixTopLevelWindowFlags函数中的if语句应该添加注释说明为什么需要根据窗口类型设置不同的标志,以便其他开发者理解这一操作的目的。
  • fixTopLevelWindowFlags函数中的if语句应该添加注释说明为什么需要根据窗口类型设置不同的标志,以便其他开发者理解这一操作的目的。
  • fixTopLevelWindowFlags函数中的if语句应该添加注释说明为什么需要根据窗口类型设置不同的标志,以便其他开发者理解这一操作的目的。
  • fixTopLevelWindowFlags函数中的if语句应该添加注释说明为什么需要根据窗口类型设置不同的标志,以便其他开发者理解这一操作的目的。
  • fixTopLevelWindowFlags函数中的if语句应该添加注释说明为什么需要根据窗口类型设置不同的标志,以便其他开发者理解这一操作的目的。
  • fixTopLevelWindowFlags函数中的if语句应该添加注释说明为什么需要根据窗口类型设置不同的标志,以便其他开发者理解这一操作的目的。
  • fixTopLevelWindowFlags函数中的if语句应该添加注释说明为什么需要根据窗口类型设置不同的标志,以便其他开发者理解这一操作的目的。
  • fixTopLevelWindowFlags函数中的if语句应该添加注释说明为什么需要根据窗口类型设置不同的标志,以便其他开发者理解这一操作的目的。
  • fixTopLevelWindowFlags函数中的if语句应该添加注释说明为什么需要根据窗口类型设置不同的标志,以便其他开发者理解这一操作的目的。
  • fixTopLevelWindowFlags函数中的if语句应该添加注释说明为什么需要根据窗口类型设置不同的标志,以便其他开发者理解这一操作的目的。
  • fixTopLevelWindowFlags函数中的if语句应该添加注释说明为什么需要

@kegechen kegechen added the hold-on should not merge now label Sep 11, 2024
@kegechen
Copy link
Contributor

too many rebot prs

@kegechen kegechen closed this Sep 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

hold-on should not merge now

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants