Skip to content

chore: 增加应用内通知动画 - #592

Merged
Whale107 merged 1 commit into
linuxdeepin:dev/animationfrom
Whale107:message
Aug 22, 2024
Merged

chore: 增加应用内通知动画#592
Whale107 merged 1 commit into
linuxdeepin:dev/animationfrom
Whale107:message

Conversation

@Whale107

@Whale107 Whale107 commented Aug 5, 2024

Copy link
Copy Markdown
Contributor

增加应用内通知动画

Log:

@deepin-ci-robot

Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: Whale107

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-ci-robot added a commit to linuxdeepin/dtk6widget that referenced this pull request Aug 5, 2024
Synchronize source files from linuxdeepin/dtkwidget.

Source-pull-request: linuxdeepin/dtkwidget#592
@deepin-bot

deepin-bot Bot commented Aug 5, 2024

Copy link
Copy Markdown
Contributor

Doc Check bot
🟡 some documents missing!

File Line Symbol
include/widgets/dmessagemanager.h 29 DMessageManager * DMessageManager::instance
include/widgets/dmessagemanager.h 32 void DMessageManager::sendMessage
include/widgets/dmessagemanager.h 33 void Dtk::Widget::DMessageManager::sendMessage
include/widgets/dmessagemanager.h 34 bool DMessageManager::setContentMargens
src/widgets/private/dmessagemanager_p.h 23 QPropertyAnimation* Dtk::Widget::DMessageManagerPrivate::m_aniGeometry
src/widgets/private/dmessagemanager_p.h 24 QPropertyAnimation* Dtk::Widget::DMessageManagerPrivate::m_aniOpacity
src/widgets/private/dmessagemanager_p.h 25 QParallelAnimationGroup* Dtk::Widget::DMessageManagerPrivate::m_aniGroup
src/widgets/private/dmessagemanager_p.h 26 QLabel* Dtk::Widget::DMessageManagerPrivate::m_label
src/widgets/private/dmessagemanager_p.h 21 Dtk::Widget::DMessageManagerPrivate::DMessageManagerPrivate
src/widgets/private/dmessagemanager_p.h 28 Dtk::Widget::DMessageManagerPrivate::D_DECLARE_PUBLIC
src/widgets/private/dmessagemanager_p.h 18 Dtk::Widget::DMessageManagerPrivate

deepin-ci-robot added a commit to linuxdeepin/dtk6widget that referenced this pull request Aug 5, 2024
Synchronize source files from linuxdeepin/dtkwidget.

Source-pull-request: linuxdeepin/dtkwidget#592
增加应用内通知动画

Log:
deepin-ci-robot added a commit to linuxdeepin/dtk6widget that referenced this pull request Aug 5, 2024
Synchronize source files from linuxdeepin/dtkwidget.

Source-pull-request: linuxdeepin/dtkwidget#592
@deepin-ci-robot

Copy link
Copy Markdown
Contributor

deepin pr auto review

关键摘要:

  • DFloatingMessage类的init()方法中,使用了connect函数来连接动画组的finished信号,但没有提供槽函数。这可能导致槽函数未连接,无法处理动画完成后的逻辑。
  • DMessageManagerPrivate类的构造函数中,m_aniGroup的初始化应该包括QPropertyAnimation::DeleteWhenStopped特性。
  • DMessageManagerPrivate类的setContentMargins方法中,应该检查content是否为nullptr,以避免潜在的空指针解引用。
  • DMessageManagerPrivate类的eventFilter方法中,对于event类型为QEvent::ChildRemoved的情况,应该添加对watched是否为nullptr的检查,以避免潜在的空指针解引用。
  • DMessageManagerPrivate类的drawPrimitive方法中,使用Qt::NoPen代替了QPen,但没有提供QPen的初始化参数,这可能会导致绘制的矩形没有边框。
  • DMessageManagerPrivate类的drawPrimitive方法中,对于peDStyle::PrimitiveRect的情况,使用了frameRadius变量,但没有提供该变量的初始化。
  • DMessageManagerPrivate类的drawPrimitive方法中,对于peDStyle::PrimitiveRoundedRect的情况,使用了frameRadius变量,但没有提供该变量的初始化。
  • DMessageManagerPrivate类的drawPrimitive方法中,对于peDStyle::PrimitivePath的情况,使用了frameRadius变量,但没有提供该变量的初始化。
  • DMessageManagerPrivate类的drawPrimitive方法中,对于peDStyle::PrimitiveImage的情况,使用了frameRadius变量,但没有提供该变量的初始化。
  • DMessageManagerPrivate类的drawPrimitive方法中,对于peDStyle::PrimitiveText的情况,使用了frameRadius变量,但没有提供该变量的初始化。
  • DMessageManagerPrivate类的drawPrimitive方法中,对于peDStyle::PrimitiveCustom的情况,使用了frameRadius变量,但没有提供该变量的初始化。

是否建议立即修改:

@Whale107
Whale107 merged commit ed80342 into linuxdeepin:dev/animation Aug 22, 2024
Whale107 added a commit to Whale107/dtkwidget that referenced this pull request Sep 23, 2024
deepin-bot Bot pushed a commit that referenced this pull request Sep 23, 2024
增加应用内通知动画

Log:
FeiWang1119 pushed a commit to linuxdeepin/dtk6widget that referenced this pull request Sep 25, 2024
Synchronize source files from linuxdeepin/dtkwidget.

Source-pull-request: linuxdeepin/dtkwidget#592
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.

2 participants