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

ProgressBar bug #37

Closed
Sally-Xu opened this issue Feb 22, 2014 · 9 comments
Closed

ProgressBar bug #37

Sally-Xu opened this issue Feb 22, 2014 · 9 comments
Labels

Comments

@Sally-Xu
Copy link
Contributor

  1. Progress Bar control missing Default Style. When using DefaultTheme, I can only see a blank box without Progress value. using MetroTheme, I can see the value.

  2. Setting Width on the ProgressBar make the Progress to show 100% regardless the current Value.

  3. Binding only works when set Mode=TwoWay, We should only need one way binding for ProgressBar as the control is a readonly control.

  4. Got "0x800a139e - JavaScript runtime error: Exception: UpdateLayout has entered infinite loop and has been aborted." error when setting ProgressBar value 100. This error can be seen on the demo page when using Metro Style, and setting 100 on the numericbox:
    http://faydedemo.azurewebsites.net/default.html#Slider

@Sally-Xu
Copy link
Contributor Author

Testing XAML:

<controls:NumericUpDown x:Name="SliderValue" DecimalPlaces="0" Increment="1" Value="50" />
<ProgressBar Value="{Binding ElementName=SliderValue, Path=Value}" Height="20" Margin="0,5" Width="300"/>

@BSick7 BSick7 added the bug label Feb 22, 2014
@BSick7
Copy link
Member

BSick7 commented Feb 22, 2014

This testing xaml is for NumericUpDown instead of ProgressBar.

@Sally-Xu
Copy link
Contributor Author

  1. Looks like the DefaultStyle is not missing, but for some reason it doesn't show the initial value until the Value changes (using Metro Theme doesn't have this problem). Test with this XAML:
    <ProgressBar Value="50" Height="20" />

@Sally-Xu
Copy link
Contributor Author

  1. Setting Width on the ProgressBar makes the Progress to show as 100% regardless the current Value.
    Testing XAML:
    <ProgressBar Value="30" Height="20" Width="200" Margin="0,5" />

@Sally-Xu
Copy link
Contributor Author

  1. Binding only works when set Mode=TwoWay on the ProgressBar, We should only need one way binding for ProgressBar as the control is a readonly control. In this testing I'm using ProgressValue NumericUpDown Control to enter the progress value. I also tested binding the ProgressBar.Value to a ViewModel's Progress property which gave me the same result:

Testing XAML:
<controls:NumericUpDown x:Name="ProgressValue" DecimalPlaces="0" Increment="1" Value="50" />
this XAML one doesn't work:

<ProgressBar Value="{Binding ElementName=ProgressValue, Path=Value}" Height="20" Margin="0,5" />

this XAML works:
<ProgressBar Value="{Binding ElementName=ProgressValue, Path=Value, Mode=TwoWay}" Height="20" Margin="0,5" />

@Sally-Xu
Copy link
Contributor Author

  1. Using the same testing XAML in case 3. Make sure to use Metro Style. I don't get the error when using Default style.

@BSick7
Copy link
Member

BSick7 commented Feb 24, 2014

Should be fixed. Please retry.

@BSick7 BSick7 closed this as completed Feb 24, 2014
@Sally-Xu
Copy link
Contributor Author

Number 3 is still not fixed, still need to set Binding mode = TwoWay. All others are working as expected.

@BSick7
Copy link
Member

BSick7 commented Feb 24, 2014

Fixed binding. Coercing of Maximum and Value in RangeBase was clearing the binding expression. Pushed nuget package 0.9.8.8.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants