-
Notifications
You must be signed in to change notification settings - Fork 39
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
chore: dci icon contorl support animation #362
Conversation
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: kegechen 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 |
Synchronize source files from linuxdeepin/dtkdeclarative. Source-pull-request: linuxdeepin/dtkdeclarative#362
Doc Check bot
|
@@ -31,11 +72,10 @@ void DQuickDciIconImageItemPrivate::maybeUpdateUrl() | |||
return DQuickIconImagePrivate::maybeUpdateUrl(); | |||
} | |||
|
|||
QUrl url; | |||
url.setScheme(QLatin1String("image")); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
都不走imageprovider了么?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
player 加载也可以改成异步。
Synchronize source files from linuxdeepin/dtkdeclarative. Source-pull-request: linuxdeepin/dtkdeclarative#362
Synchronize source files from linuxdeepin/dtkdeclarative. Source-pull-request: linuxdeepin/dtkdeclarative#362
Synchronize source files from linuxdeepin/dtkdeclarative. Source-pull-request: linuxdeepin/dtkdeclarative#362
_._20240725113815.mp4 |
Synchronize source files from linuxdeepin/dtkdeclarative. Source-pull-request: linuxdeepin/dtkdeclarative#362
Synchronize source files from linuxdeepin/dtkdeclarative. Source-pull-request: linuxdeepin/dtkdeclarative#362
src/private/dquickdciiconimage.cpp
Outdated
QString iconPath = findDciIconPath(parentPriv->imageItem->name(), appIconThemeName()); | ||
|
||
// 防止频繁构造 dciicon | ||
if (player->property("_d_dci_icon_path").toString() != iconPath) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
可以给dciicon加个判断的接口,
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
这个每次都是重新构造一个 DDciIcon
, 只能说在 DQuickDciIconImageItemPrivate
加一个 iconPathCache
Synchronize source files from linuxdeepin/dtkdeclarative. Source-pull-request: linuxdeepin/dtkdeclarative#362
Synchronize source files from linuxdeepin/dtkdeclarative. Source-pull-request: linuxdeepin/dtkdeclarative#362
Synchronize source files from linuxdeepin/dtkdeclarative. Source-pull-request: linuxdeepin/dtkdeclarative#362
update checkbox / radio / switch dci icons switch: disable hover and set static image before after play animation(use timer for now)
button group ( ButtonBox/ToolButton)
deepin pr auto reviewMake sure to use the correct icon name for switch buttons |
Synchronize source files from linuxdeepin/dtkdeclarative. Source-pull-request: linuxdeepin/dtkdeclarative#362
update checkbox / radio dci icon
linuxdeepin/dtkgui#247