Skip to content

fix: temporarily disable flickable bounds behavior#750

Merged
deepin-bot[bot] merged 1 commit into
linuxdeepin:masterfrom
wjyrich:fix-bug-356899
Apr 21, 2026
Merged

fix: temporarily disable flickable bounds behavior#750
deepin-bot[bot] merged 1 commit into
linuxdeepin:masterfrom
wjyrich:fix-bug-356899

Conversation

@wjyrich
Copy link
Copy Markdown
Contributor

@wjyrich wjyrich commented Apr 20, 2026

  1. Commented out the boundsBehavior property setting in the GridView
  2. This change disables the "StopAtBounds" behavior that was preventing scrolling beyond content boundaries
  3. The property is temporarily disabled for testing purposes or to allow smoother scrolling behavior

Influence:

  1. Test scrolling behavior in the GridView to ensure it can scroll beyond content boundaries
  2. Verify that the UI remains responsive and doesn't exhibit unexpected behavior
  3. Check if any other scrolling-related properties need adjustment
  4. Test with different content sizes to ensure proper functionality
  5. Monitor performance impact of removing bounds constraints

fix: 临时禁用可滑动组件的边界行为

  1. 注释掉了 GridView 中的 boundsBehavior 属性设置
  2. 此更改禁用了阻止滚动超出内容边界的 "StopAtBounds" 行为
  3. 暂时禁用此属性用于测试目的或实现更平滑的滚动行为

Influence:

  1. 测试 GridView 中的滚动行为,确保可以滚动超出内容边界
  2. 验证 UI 保持响应且不会出现意外行为
  3. 检查是否需要调整其他与滚动相关的属性
  4. 使用不同大小的内容进行测试以确保功能正常
  5. 监控移除边界约束对性能的影响

PMS: BUG-356899

BUG-343525 修改后未复现,后续提BUG继续追踪。

1. Commented out the boundsBehavior property setting in the GridView
2. This change disables the "StopAtBounds" behavior that was preventing
scrolling beyond content boundaries
3. The property is temporarily disabled for testing purposes or to allow
smoother scrolling behavior

Influence:
1. Test scrolling behavior in the GridView to ensure it can scroll
beyond content boundaries
2. Verify that the UI remains responsive and doesn't exhibit unexpected
behavior
3. Check if any other scrolling-related properties need adjustment
4. Test with different content sizes to ensure proper functionality
5. Monitor performance impact of removing bounds constraints

fix: 临时禁用可滑动组件的边界行为

1. 注释掉了 GridView 中的 boundsBehavior 属性设置
2. 此更改禁用了阻止滚动超出内容边界的 "StopAtBounds" 行为
3. 暂时禁用此属性用于测试目的或实现更平滑的滚动行为

Influence:
1. 测试 GridView 中的滚动行为,确保可以滚动超出内容边界
2. 验证 UI 保持响应且不会出现意外行为
3. 检查是否需要调整其他与滚动相关的属性
4. 使用不同大小的内容进行测试以确保功能正常
5. 监控移除边界约束对性能的影响

PMS: BUG-356899

BUG-343525 修改后未复现,后续提BUG继续追踪。
Copy link
Copy Markdown

@sourcery-ai sourcery-ai Bot left a comment

Choose a reason for hiding this comment

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

Sorry @wjyrich, you have reached your weekly rate limit of 500000 diff characters.

Please try again later or upgrade to continue using Sourcery

Copy link
Copy Markdown
Member

@BLumia BLumia left a comment

Choose a reason for hiding this comment

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

印象里之前说的是不回弹。。。

@deepin-ci-robot
Copy link
Copy Markdown

[APPROVALNOTIFIER] This PR is NOT APPROVED

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

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

@wjyrich
Copy link
Copy Markdown
Contributor Author

wjyrich commented Apr 21, 2026

/forcemerge

@deepin-bot
Copy link
Copy Markdown

deepin-bot Bot commented Apr 21, 2026

This pr force merged! (status: blocked)

@deepin-bot deepin-bot Bot merged commit 738f41f into linuxdeepin:master Apr 21, 2026
7 checks passed
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.

3 participants