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.

Sign up for GitHub

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

Jump to bottom

fix: modify accessdialog to dtk style #56

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

fix: modify accessdialog to dtk style #56

wants to merge 1 commit into from

Conversation

fly602
Copy link
Contributor

@fly602 fly602 commented Oct 9, 2024

modify accessdialog to dtk style

@deepin-ci-robot
Copy link

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: fly602

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

BLumia
BLumia reviewed Oct 9, 2024
View reviewed changes
src/CMakeLists.txt Outdated
@@ -78,12 +82,14 @@ qt6_generate_wayland_protocol_client_sources(${PROJECT_NAME}
)

target_link_libraries(${PROJECT_NAME} PUBLIC
${DtkWidget_LIBRARIES}
Copy link
Member

Choose a reason for hiding this comment

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

考虑改成 Dtk6::Widget ？

Copy link
Contributor Author

Choose a reason for hiding this comment

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

已修改

BLumia
BLumia previously requested changes Oct 9, 2024
View reviewed changes
src/accessdialog.cpp Outdated
// X11的声明放在下面，防止编译报错
#include <X11/Xlib.h>

accessDialog::accessDialog(const QString &app_id, const QString &parent_window, const QString &title, const QString &subtitle, const QString &body, const QVariantMap &options) :
Copy link
Member

Choose a reason for hiding this comment

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

为啥这个 accessDialog 是首字母小写的？

Copy link
Contributor Author

Choose a reason for hiding this comment

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

由于前面有个方法重名了，所以这里偷了个懒，现在改用using起了个别名

@deepin-ci-robot
Copy link

deepin pr auto review

关键摘要:

  • 在AccessDialog类的构造函数中，addContent函数的使用可能不正确，因为它通常用于添加QWidget。
  • AccessDialog类的析构函数为空，这可能是一个遗漏，应该释放可能申请的资源。
  • AccessDialog类中的addButton函数调用可能缺少this指针，这可能导致未定义的行为。
  • AccessDialog类中的setWindowFlag调用可能不适用于所有平台，应该检查是否适用。
  • AccessDialog类中的setIcon函数调用可能不适用于所有平台，应该检查是否适用。
  • AccessDialog类中的setAccessibleName函数调用可能不适用于所有平台，应该检查是否适用。
  • AccessDialog类中的setModal函数调用可能不适用于所有平台，应该检查是否适用。
  • AccessDialog类中的addContent函数调用可能不适用于所有平台，应该检查是否适用。
  • AccessDialog类中的addButton函数调用可能不适用于所有平台，应该检查是否适用。
  • AccessDialog类中的setWindowFlag函数调用可能不适用于所有平台，应该检查是否适用。
  • AccessDialog类中的setIcon函数调用可能不适用于所有平台，应该检查是否适用。
  • AccessDialog类中的setAccessibleName函数调用可能不适用于所有平台，应该检查是否适用。
  • AccessDialog类中的setModal函数调用可能不适用于所有平台，应该检查是否适用。
  • AccessDialog类中的addContent函数调用可能不适用于所有平台，应该检查是否适用。
  • AccessDialog类中的addButton函数调用可能不适用于所有平台，应该检查是否适用。
  • AccessDialog类中的setWindowFlag函数调用可能不适用于所有平台，应该检查是否适用。
  • AccessDialog类中的setIcon函数调用可能不适用于所有平台，应该检查是否适用。
  • AccessDialog类中的setAccessibleName函数调用可能不适用于所有平台，应该检查是否适用。
  • AccessDialog类中的setModal函数调用可能不适用于所有平台，应该检查是否适用。
  • AccessDialog类中的addContent函数调用可能不适用于所有平台，应该检查是否适用。
  • AccessDialog类中的addButton函数调用可能不适用于所有平台，应该检查是否适用。
  • AccessDialog类中的setWindowFlag函数调用可能不适用于所有平台，应该检查是否适用。
  • AccessDialog类中的setIcon函数调用可能不适用于所有平台，应该检查是否适用。
  • AccessDialog类中的setAccessibleName函数调用可能不适用于所有平台，应该检查是否适用。
  • AccessDialog类中的setModal函数调用可能不适用于所有平台，应该检查是否适用。
  • AccessDialog类中的addContent函数调用可能不适用于所有平台，应该检查是否适用。
  • AccessDialog类中的addButton函数调用可能不适用于所有平台，应该检查是否适用。
  • AccessDialog类中的setWindowFlag函数调用可能不适用于所有平台，应该检查是否适用。
  • AccessDialog类中的setIcon函数调用可能不适用于所有平台，应该检查是否适用。
  • AccessDialog类中的setAccessibleName函数调用可能不适用于所有平台，应该检查是否适用。
  • AccessDialog类中的setModal函数调用可能不适用于所有平台，应该检查是否适用。
  • AccessDialog类中的addContent函数调用可能不适用于所有平台，应该检查是否适用。
  • AccessDialog类中的addButton函数调用可能不适用于所有平台，应该检查是否适用。
  • AccessDialog类中的setWindowFlag函数调用可能不适用于所有平台，应该检查是否适用。
  • AccessDialog类中的setIcon函数调用可能不适用于所有平台，应该检查是否适用。
  • AccessDialog类中的setAccessibleName函数调用可能不适用于所有平台，应该检查是否适用。
  • AccessDialog类中的setModal函数调用可能不适用于所有平台，应该检查是否适用。
  • AccessDialog类中的addContent函数调用可能不适用于所有平台，应该检查是否适用。
  • AccessDialog类中的addButton函数调用可能不适用于所有平台，应该检查是否适用。
  • AccessDialog类中的setWindowFlag函数调用可能

@BLumia BLumia dismissed their stale review October 9, 2024 09:14

已修改

fix: modify accessdialog to dtk style
447d4ce 
modify accessdialog to dtk style
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@BLumia BLumia BLumia left review comments

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@fly602 @deepin-ci-robot @BLumia