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.

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: add some interfaces about personalization #258

Closed
wants to merge 1 commit into from

Conversation

FeiWang1119
Copy link
Contributor

add some interfaces about personalization

Log:

@deepin-ci-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: FeiWang1119

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

deepin-ci-robot added a commit to linuxdeepin/dtk6gui that referenced this pull request Oct 10, 2024
Synchronize source files from linuxdeepin/dtkgui.

Source-pull-request: linuxdeepin/dtkgui#258
deepin-ci-robot added a commit to linuxdeepin/dtk6gui that referenced this pull request Oct 10, 2024
Synchronize source files from linuxdeepin/dtkgui.

Source-pull-request: linuxdeepin/dtkgui#258
deepin-ci-robot added a commit to linuxdeepin/dtk6gui that referenced this pull request Oct 11, 2024
Synchronize source files from linuxdeepin/dtkgui.

Source-pull-request: linuxdeepin/dtkgui#258
deepin-ci-robot added a commit to linuxdeepin/dtk6gui that referenced this pull request Oct 11, 2024
Synchronize source files from linuxdeepin/dtkgui.

Source-pull-request: linuxdeepin/dtkgui#258
deepin-ci-robot added a commit to linuxdeepin/dtk6gui that referenced this pull request Oct 12, 2024
Synchronize source files from linuxdeepin/dtkgui.

Source-pull-request: linuxdeepin/dtkgui#258
deepin-ci-robot added a commit to linuxdeepin/dtk6gui that referenced this pull request Oct 12, 2024
Synchronize source files from linuxdeepin/dtkgui.

Source-pull-request: linuxdeepin/dtkgui#258
deepin-ci-robot added a commit to linuxdeepin/dtk6gui that referenced this pull request Oct 12, 2024
Synchronize source files from linuxdeepin/dtkgui.

Source-pull-request: linuxdeepin/dtkgui#258
deepin-ci-robot added a commit to linuxdeepin/dtk6gui that referenced this pull request Oct 12, 2024
Synchronize source files from linuxdeepin/dtkgui.

Source-pull-request: linuxdeepin/dtkgui#258
deepin-ci-robot added a commit to linuxdeepin/dtk6gui that referenced this pull request Oct 12, 2024
Synchronize source files from linuxdeepin/dtkgui.

Source-pull-request: linuxdeepin/dtkgui#258
deepin-ci-robot added a commit to linuxdeepin/dtk6gui that referenced this pull request Oct 12, 2024
Synchronize source files from linuxdeepin/dtkgui.

Source-pull-request: linuxdeepin/dtkgui#258
deepin-ci-robot added a commit to linuxdeepin/dtk6gui that referenced this pull request Oct 12, 2024
Synchronize source files from linuxdeepin/dtkgui.

Source-pull-request: linuxdeepin/dtkgui#258
deepin-ci-robot added a commit to linuxdeepin/dtk6gui that referenced this pull request Oct 12, 2024
Synchronize source files from linuxdeepin/dtkgui.

Source-pull-request: linuxdeepin/dtkgui#258
@kegechen kegechen requested review from justforlxz and 18202781743 and removed request for justforlxz October 12, 2024 06:33
deepin-ci-robot added a commit to linuxdeepin/dtk6gui that referenced this pull request Oct 12, 2024
Synchronize source files from linuxdeepin/dtkgui.

Source-pull-request: linuxdeepin/dtkgui#258
deepin-ci-robot added a commit to linuxdeepin/dtk6gui that referenced this pull request Oct 12, 2024
Synchronize source files from linuxdeepin/dtkgui.

Source-pull-request: linuxdeepin/dtkgui#258
src/CMakeLists.txt Outdated Show resolved Hide resolved
src/kernel/dplatformtheme.cpp Outdated Show resolved Hide resolved
@FeiWang1119 FeiWang1119 requested a review from zccrs October 14, 2024 12:51
src/CMakeLists.txt Outdated Show resolved Hide resolved
src/kernel/dguiapplicationhelper.cpp Outdated Show resolved Hide resolved
src/kernel/dplatformhandle.cpp Outdated Show resolved Hide resolved
src/kernel/dplatformhandle.cpp Outdated Show resolved Hide resolved
src/kernel/dplatformhandle.cpp Outdated Show resolved Hide resolved
src/kernel/dplatformhandle.cpp Outdated Show resolved Hide resolved
src/kernel/dplatformhandle.cpp Outdated Show resolved Hide resolved
src/kernel/dplatformtheme.cpp Outdated Show resolved Hide resolved
deepin-ci-robot added a commit to linuxdeepin/dtk6gui that referenced this pull request Oct 15, 2024
Synchronize source files from linuxdeepin/dtkgui.

Source-pull-request: linuxdeepin/dtkgui#258
deepin-ci-robot added a commit to linuxdeepin/dtk6gui that referenced this pull request Oct 15, 2024
Synchronize source files from linuxdeepin/dtkgui.

Source-pull-request: linuxdeepin/dtkgui#258
deepin-ci-robot added a commit to linuxdeepin/dtk6gui that referenced this pull request Oct 15, 2024
Synchronize source files from linuxdeepin/dtkgui.

Source-pull-request: linuxdeepin/dtkgui#258
deepin-ci-robot added a commit to linuxdeepin/dtk6gui that referenced this pull request Oct 15, 2024
Synchronize source files from linuxdeepin/dtkgui.

Source-pull-request: linuxdeepin/dtkgui#258
deepin-ci-robot added a commit to linuxdeepin/dtk6gui that referenced this pull request Oct 15, 2024
Synchronize source files from linuxdeepin/dtkgui.

Source-pull-request: linuxdeepin/dtkgui#258
@deepin-ci-robot
Copy link
Contributor

deepin pr auto review

Copy link
Member

Choose a reason for hiding this comment

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

先不要作为public接口,先作为private的,不对外可见

include/kernel/dplatforminterface.h Outdated Show resolved Hide resolved
virtual int windowRadius() const;
virtual void setWindowRadius(int windowRadius);
virtual QByteArray fontName() const;
virtual void setFontName(const QByteArray &fontName);
Copy link
Member

Choose a reason for hiding this comment

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

这种系统全局类的配置的修改不在 dtkgui 里提供,dtkgui里只读,不实现修改的功能。

Copy link
Member

Choose a reason for hiding this comment

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

就算要做修改的功能,也应该是在 dtksystemsettings 模块里做。

virtual ~DPlatformInterface();

virtual int windowRadius() const;
virtual void setWindowRadius(int windowRadius);
Copy link
Member

Choose a reason for hiding this comment

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

如果是针对某个窗口的,那可以留着设置接口

Copy link
Member

Choose a reason for hiding this comment

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

作为一个单独的cmake模块放到一个子目录里,目录以 plugins/platform 开头,比如是 plugins/platform/xcb

Copy link
Member

Choose a reason for hiding this comment

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

同样的也应该是 private 的,不对外

if (contextWindow->noTitlebar() == enable)
return true;
contextWindow->setNoTitlebar(enable);
#ifndef DTK_DISABLE_TREELAND
Copy link
Member

Choose a reason for hiding this comment

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

这部分为什么没有拆到 DPlatformHandle 里去?

@@ -1209,6 +1203,12 @@ WId DPlatformHandle::windowLeader()

void DPlatformHandle::setWindowRadius(int windowRadius)
{
#ifndef DTK_DISABLE_TREELAND
if (DGuiApplicationHelper::testAttribute(DGuiApplicationHelper::IsTreelandPlatform)) {
Copy link
Member

Choose a reason for hiding this comment

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

同上

{
}

int DPlatformInterface::windowRadius() const
Copy link
Member

Choose a reason for hiding this comment

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

可以直接是纯虚函数

@@ -19,7 +26,6 @@ DGUI_BEGIN_NAMESPACE
DPlatformThemePrivate::DPlatformThemePrivate(Dtk::Gui::DPlatformTheme *qq)
: DNativeSettingsPrivate(qq, QByteArrayLiteral("/deepin/palette"))
{

Copy link
Member

Choose a reason for hiding this comment

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

为啥删了

: QWaylandClientExtensionTemplate<PersonalizationManager>(1)
{
if (QGuiApplication::platformName() == QLatin1String("wayland")) {
QtWaylandClient::QWaylandIntegration *waylandIntegration = static_cast<QtWaylandClient::QWaylandIntegration *>(QGuiApplicationPrivate::platformIntegration());
Copy link
Member

Choose a reason for hiding this comment

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

Qt6版本不是这样使用了吧,它不是有了 QNativeInterface 这种东西

deepin-ci-robot added a commit to linuxdeepin/dtk6gui that referenced this pull request Oct 17, 2024
Synchronize source files from linuxdeepin/dtkgui.

Source-pull-request: linuxdeepin/dtkgui#258
deepin-ci-robot added a commit to linuxdeepin/dtk6gui that referenced this pull request Oct 17, 2024
Synchronize source files from linuxdeepin/dtkgui.

Source-pull-request: linuxdeepin/dtkgui#258
@FeiWang1119 FeiWang1119 marked this pull request as draft October 23, 2024 05:22
@deepin-bot
Copy link
Contributor

deepin-bot bot commented Oct 31, 2024

TAG Bot

New tag: 5.7.1
DISTRIBUTION: unstable
Suggest: synchronizing this PR through rebase #256

@deepin-bot
Copy link
Contributor

deepin-bot bot commented Nov 15, 2024

TAG Bot

New tag: 5.7.2
DISTRIBUTION: unstable
Suggest: synchronizing this PR through rebase #271

@deepin-bot
Copy link
Contributor

deepin-bot bot commented Nov 22, 2024

TAG Bot

New tag: 5.7.3
DISTRIBUTION: unstable
Suggest: synchronizing this PR through rebase #273

deepin-ci-robot added a commit to linuxdeepin/dtk6gui that referenced this pull request Nov 28, 2024
Synchronize source files from linuxdeepin/dtkgui.

Source-pull-request: linuxdeepin/dtkgui#258
add window platform interface

Log: as title.
deepin-ci-robot added a commit to linuxdeepin/dtk6gui that referenced this pull request Nov 28, 2024
Synchronize source files from linuxdeepin/dtkgui.

Source-pull-request: linuxdeepin/dtkgui#258
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