Skip to content
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.

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

Listbox Scrollbar issue #247

Closed
Ronmenator opened this issue Feb 18, 2016 · 4 comments
Closed

Listbox Scrollbar issue #247

Ronmenator opened this issue Feb 18, 2016 · 4 comments

Comments

@Ronmenator
Copy link
Contributor

Hi,

Attached is an example of a Listbox scrollbar problem.

You have a page with two Columns.

In the left Column, you have a Listbox that has a list of objects. Each object has a list of children.
In the right Column, you have another Listbox whose ItemsSource is bound to the first Listbox's SelectedItem.Children.

As you swap between the Items in the Left Column Listbox, the right Column Listbox displays the children of the Selected item.

The problem arises when one of the elements on the left has a large number of children so that a scrollbar is visible on the Right Listbox. Swapping back to an item that does not require a Scrollbar on the right causes the ListBox to get a scrollbar that is massive and scrolled down to the bottom.

See the attached code. I am not close to something to generate a pull request with.

ListBoxScroll.zip

@BSick7
Copy link
Member

BSick7 commented Feb 20, 2016

I believe this is intended by the authors of the metro theme.
They gave the scrollbars height/width of 18; however, the visuals never takes up 18px.

@BSick7
Copy link
Member

BSick7 commented Feb 20, 2016

I'm going to close this, but if you feel strongly about changing the theme, we can reopen.

@BSick7 BSick7 closed this as completed Feb 20, 2016
@Ronmenator
Copy link
Contributor Author

Ok, to solve the scrolling to the bottom issue, I just have to remove the width and height of the scrollbar.

@BSick7
Copy link
Member

BSick7 commented Feb 20, 2016

I misunderstood the issue. Reopening.

@BSick7 BSick7 reopened this Feb 20, 2016
@BSick7 BSick7 closed this as completed in ee2b99d Feb 20, 2016
BSick7 added a commit that referenced this issue Feb 20, 2016
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

No branches or pull requests

2 participants