Skip to content

feat: support base64 format for icons#286

Merged
18202781743 merged 1 commit into
linuxdeepin:masterfrom
18202781743:base64-icon
Jan 29, 2024
Merged

feat: support base64 format for icons#286
18202781743 merged 1 commit into
linuxdeepin:masterfrom
18202781743:base64-icon

Conversation

@18202781743

Copy link
Copy Markdown
Contributor

We don't use ImageProvider to support base64 image, although it can
use Qt's pixmap cache, the cache will use url as a part of the cache key.

  We don't use ImageProvider to support base64 image, although it can
use Qt's pixmap cache, the cache will use url as a part of the cache key.
deepin-ci-robot added a commit to linuxdeepin/dtk6declarative that referenced this pull request Jan 23, 2024
Synchronize source files from linuxdeepin/dtkdeclarative.

Source-pull-request: linuxdeepin/dtkdeclarative#286
@deepin-bot

deepin-bot Bot commented Jan 23, 2024

Copy link
Copy Markdown
Contributor

Doc Check bot
🟡 some documents missing!

File Line Symbol
src/private/dquickdciiconimage_p_p.h 44 bool Dtk::Quick::DQuickDciIconImagePrivate::fallbackToQIcon

@18202781743

Copy link
Copy Markdown
Contributor Author

/check obs

@deepin-bot

deepin-bot Bot commented Jan 29, 2024

Copy link
Copy Markdown
Contributor

Deepin Obs Bot
Starting find obs webhook event and retrigger!

@deepin-ci-robot

Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: 18202781743, asterwyx, BLumia, zccrs

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

@18202781743
18202781743 merged commit 8a142ef into linuxdeepin:master Jan 29, 2024
18202781743 pushed a commit to linuxdeepin/dtk6declarative that referenced this pull request Jan 29, 2024
Synchronize source files from linuxdeepin/dtkdeclarative.

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

5 participants