Skip to content

[Improvement][UI] Scrolling will be applied to parent model on editor scrolled to top/bottom#11335

Merged
songjianet merged 1 commit intoapache:devfrom
hezean:editor-scroll
Aug 8, 2022
Merged

[Improvement][UI] Scrolling will be applied to parent model on editor scrolled to top/bottom#11335
songjianet merged 1 commit intoapache:devfrom
hezean:editor-scroll

Conversation

@hezean
Copy link
Contributor

@hezean hezean commented Aug 7, 2022

Purpose of the pull request

Fixes #11328 (Formally the parent model couldn't be scrolled if the mouse is positioned in the editor)

Verify this pull request

This change added tests and can be verified as follows:

8w02k-bls5k

@github-actions github-actions bot added the UI ui and front end related label Aug 7, 2022
@codecov-commenter
Copy link

codecov-commenter commented Aug 7, 2022

Codecov Report

Merging #11335 (1dfd34b) into dev (3aa9f2e) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##                dev   #11335   +/-   ##
=========================================
  Coverage     40.26%   40.26%           
  Complexity     4843     4843           
=========================================
  Files           974      974           
  Lines         37323    37323           
  Branches       4142     4142           
=========================================
  Hits          15028    15028           
  Misses        20749    20749           
  Partials       1546     1546           

📣 Codecov can now indicate which changes are the most critical in Pull Requests. Learn more

@sonarqubecloud
Copy link

sonarqubecloud bot commented Aug 7, 2022

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@songjianet songjianet added this to the 3.1.0 milestone Aug 8, 2022
@songjianet songjianet added the improvement make more easy to user or prompt friendly label Aug 8, 2022
Copy link
Member

@songjianet songjianet left a comment

Choose a reason for hiding this comment

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

LGTM

@songjianet songjianet merged commit 991414e into apache:dev Aug 8, 2022
@hezean hezean deleted the editor-scroll branch August 8, 2022 16:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

improvement make more easy to user or prompt friendly UI ui and front end related

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Improvement][UI] Mouse sliding optimization

3 participants