Skip to content

Popup size #933

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
Merged
deepin-bot merged 5 commits into from
Dec 6, 2024
Merged

Popup size #933

deepin-bot merged 5 commits into from
Dec 6, 2024

Conversation

@tsic404
Copy link
Contributor

@tsic404 tsic404 commented Dec 5, 2024 •
edited
Loading

  • 修复popup不完整问题
    问题在于QQUickWindow的contentItem尺寸和QQUickWindow没有保持一致导致的，理论上应该是一致的，但是实际出现了不一致的情况，所以临时使用了Binding将两者尺寸绑定到一起并加了FIXME

  • 修复popup缩放问题
    popup的缩放之前是通过环境变量设置过去的，但是treeland下支持动态改动缩放，导致缩放不能动态的更着变更。

    1. 去掉环境变量使用wayland 分数缩放协议来做缩放的动态调整
    2. 通过协议提交尺寸，而不是通过buffer来计算尺寸，因为output级别的缩放仅支持整数缩放而不支持分数缩放，会导致尺寸计算不太正确，所以直接使用协议设置对应的尺寸过来。依赖dde-tray-loader 对应的提交fix: popup size not right on treeland dde-tray-loader#211

yixinshark
yixinshark reviewed Dec 5, 2024
View reviewed changes
Copy link
Contributor

@yixinshark yixinshark left a comment

Choose a reason for hiding this comment

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

添加一下control中依赖dde-tray-loader-dev的版本，改了协议。

@tsic404
fix: The popup window content is incomplete
d6d1e7e 
Temporarily bind the size of contentItem together using Binding

log: as title
@tsic404
fix: protocol error caused by illegal size
1fcd263 
make popup minimumsize to 10

log: as title
@tsic404
fix: Crash caused by unsupported protocol
43ced7f 
log: as title
@tsic404
chore: adjust cmake base version to 1.99.0
c0ab52d 
log: as title
@tsic404
fix: popup scaling incorrectly
174fe9a 
Set plugin scaling via fractional scaling protocol instead of
QT environment variables

log: as title
@tsic404 tsic404 force-pushed the popupFocus branch 2 times, most recently from f3e7dcd to 174fe9a Compare December 5, 2024 11:59
@deepin-ci-robot
Copy link

deepin-ci-robot commented Dec 5, 2024

deepin pr auto review

代码审查意见：

  1. 在CMakeLists.txt文件中，版本号从0.0.34更新到1.99.0，需要确认这个版本号的更新是否符合项目的版本管理策略，并且是否与相关的文档和发布说明保持一致。

  2. 在debian/control文件中，dde-tray-loader-dev的版本号从>= 1.99.5更新到> 1.99.6，需要确认这个更新是否与依赖库的版本兼容，并且是否已经测试过新的版本。

  3. 在frame/popupwindow.cpp文件中，新增了setMinimumHeight(10);和setMinimumWidth(10);，需要确认这些最小尺寸是否满足应用的需求，并且是否会对用户界面产生负面影响。

  4. 在frame/qml/PanelPopupWindow.qml文件中，新增了两个Binding来同步root.contentItem的宽度和高度，但是注释中提到可能存在QtWayland的bug，需要进一步调查这个问题的原因和影响。

  5. 在panels/dock/CMakeLists.txt文件中，新增了fractional-scale-v1.xml文件，需要确认这个文件是否是必要的，并且是否已经测试过相关的功能。

  6. 在panels/dock/DockCompositor.qml文件中，新增了一个panelScale属性，并且与PluginScaleManager关联，需要确认这个属性是否正确地实现了 fractional scale 的功能，并且是否已经测试过相关的功能。

  7. 在panels/dock/dockpanel.cpp文件中，新增了一个devicePixelRatioChanged信号，并且在eventFilter函数中处理了QEvent::DevicePixelRatioChange事件，需要确认这个信号的实现是否正确，并且是否已经测试过相关的功能。

  8. 在panels/dock/loadtrayplugins.cpp文件中，移除了对QT_SCALE_FACTOR和D_DXCB_DISABLE_OVERRIDE_HIDPI环境变量的设置，需要确认这个改动是否会影响其他部分的代码。

  9. 在panels/dock/package/main.qml文件中，DockCompositor.panelScale的绑定表达式被修改，需要确认这个改动是否正确，并且是否已经测试过相关的功能。

  10. 在panels/dock/pluginmanagerextension.cpp和panels/dock/pluginmanagerextension_p.h文件中，新增了PluginScaleManager和PluginScale类，需要确认这些类的实现是否正确，并且是否已经测试过相关的功能。

  11. 在panels/dock/pluginmanagerextension_p.h文件中，PluginSurface和PluginPopup类的pluginSize属性被替换为height和width属性，需要确认这个改动是否正确，并且是否已经测试过相关的功能。

  12. 在panels/dock/taskmanager/treelandwindowmonitor.cpp文件中，新增了一个检查m_foreignToplevelManager->isActive()的条件判断，需要确认这个改动是否正确，并且是否已经测试过相关的功能。

  13. 在panels/dock/tray/ShellSurfaceItemProxy.qml文件中，ShellSurfaceItem的anchors.fill属性被设置，需要确认这个改动是否正确，并且是否已经测试过相关的功能。

  14. 在panels/dock/tray/package/ActionLegacyTrayPluginDelegate.qml文件中，AppletItemButton的implicitHeight和implicitWidth属性被修改，需要确认这个改动是否正确，并且是否已经测试过相关的功能。

总的来说，这些改动都需要经过充分的测试和验证，以确保它们不会引入新的问题，并且能够正确地实现预期的功能。

@deepin-ci-robot
Copy link

deepin-ci-robot commented Dec 5, 2024

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: tsic404, yixinshark

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

@tsic404
Copy link
Contributor Author

tsic404 commented Dec 6, 2024

/forcemerge

@deepin-bot
Copy link

deepin-bot bot commented Dec 6, 2024

This pr force merged! (status: behind)

@deepin-bot deepin-bot bot merged commit f691bcd into linuxdeepin:master Dec 6, 2024
7 of 10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@yixinshark yixinshark yixinshark approved these changes

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

@tsic404 @deepin-ci-robot @yixinshark