Skip to content

fix: layershell not show#360

Merged
BLumia merged 1 commit into
linuxdeepin:masterfrom
justforlxz:master
Jul 23, 2024
Merged

fix: layershell not show#360
BLumia merged 1 commit into
linuxdeepin:masterfrom
justforlxz:master

Conversation

@justforlxz

Copy link
Copy Markdown
Member

remove disable resize

Log:

remove disable resize

Log:
@deepin-ci-robot

Copy link
Copy Markdown

deepin pr auto review

关键摘要:

  • 移除了maximumWidthminimumWidth的设置,可能会影响窗口的缩放行为。
  • 移除了minimumHeight的设置,可能会导致窗口高度无法调整到最小高度。
  • 移除了maximumHeight的设置,可能会导致窗口高度无法调整到最大高度。

是否建议立即修改:

  • 是,需要确认这些更改是否符合设计需求和用户体验。如果这些设置是必要的,则需要恢复或提供替代的解决方案。

@justforlxz
justforlxz requested a review from tsic404 July 23, 2024 03:17
@BLumia

BLumia commented Jul 23, 2024

Copy link
Copy Markdown
Member

cc @18202781743

@BLumia BLumia left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

相当于 revert b7f2d43

@deepin-ci-robot

Copy link
Copy Markdown

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: BLumia, justforlxz

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

@wineee

wineee commented Jul 25, 2024

Copy link
Copy Markdown
Member

相当于 revert b7f2d43

#340 也没了

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.

4 participants