Skip to content

Bugfix/550 tree view scroll#551

Merged
pomianowski merged 1 commit intolepoco:developmentfrom
Korjam:bugfix/550-tree-view-scroll
Mar 3, 2023
Merged

Bugfix/550 tree view scroll#551
pomianowski merged 1 commit intolepoco:developmentfrom
Korjam:bugfix/550-tree-view-scroll

Conversation

@Korjam
Copy link
Copy Markdown
Contributor

@Korjam Korjam commented Feb 27, 2023

Pull request type

Please check the type of change your PR introduces:

  • Update
  • Bugfix
  • Feature
  • Code style update (formatting, renaming)
  • Refactoring (no functional changes, no api changes)
  • Build related changes
  • Documentation content changes

What is the current behavior?

Issue Number: 550

What is the new behavior?

Now TreeView is scrolling correctly when virtualization is disabled

Other information

@Korjam Korjam requested a review from pomianowski as a code owner February 27, 2023 14:05
@Korjam Korjam changed the base branch from main to development February 27, 2023 14:06
@github-actions github-actions bot added github_actions Pull requests that update GitHub Actions code PR Pull request labels Feb 27, 2023
CornerRadius="4">
<ScrollViewer
CanContentScroll="{TemplateBinding ScrollViewer.CanContentScroll}"
Name="_tv_scrollviewer_"
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.

Weird name for XAML element. Something like ItemsPresenterScrollViewer or PART_ScrollViewer (if used by the class) would be better.

<Setter
TargetName="_tv_scrollviewer_"
Property="CanContentScroll"
Value="true"/>
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.

Please use capital True for XAML.

<ScrollViewer
CanContentScroll="{TemplateBinding ScrollViewer.CanContentScroll}"
Name="_tv_scrollviewer_"
CanContentScroll="false"
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.

Please use capital False for XAML.

@github-actions github-actions bot removed github_actions Pull requests that update GitHub Actions code PR Pull request labels Mar 3, 2023
@pomianowski pomianowski merged commit 5ae2524 into lepoco:development Mar 3, 2023
@Korjam Korjam deleted the bugfix/550-tree-view-scroll branch March 3, 2023 14:55
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Mar 3, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

TreeView scrolling is not working when virtualization is not enabled.

2 participants