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

MudExpansionPanel crashes with two bind-IsExpanded #1153

Closed
poteb opened this issue Mar 10, 2021 · 1 comment
Closed

MudExpansionPanel crashes with two bind-IsExpanded #1153

poteb opened this issue Mar 10, 2021 · 1 comment
Labels
bug Something does not work as intended/expected fixed Fix has been merged
Milestone

Comments

@poteb
Copy link
Contributor

poteb commented Mar 10, 2021

The code
`
<MudExpansionPanel Text="Panel 1" @bind-IsExpanded="IsPanel1Open">
Content

<MudExpansionPanel Text="Panel 2" @bind-IsExpanded="IsPanel2Open">
Content

@code
{
bool IsPanel1Open;
bool IsPanel2Open;
}`

Steps to reproduce

  1. Open one panel and keep it open.
  2. Open the other panel. Page crashes because of endless loop.

TryMudBlazor
https://try.mudblazor.com/snippet/waQluHFaSEEeJTJG

@poteb poteb added the bug Something does not work as intended/expected label Mar 10, 2021
@poteb
Copy link
Contributor Author

poteb commented Mar 10, 2021

I found a way to mitigate the bug, added MultiExpansion="true" to MudExpansionPanels.
But I think it's still a bug that should be looked at.

@mikes-gh mikes-gh added the fixed Fix has been merged label Mar 10, 2021
@mikes-gh mikes-gh added this to the 5.0.6 milestone Mar 10, 2021
mikes-gh pushed a commit that referenced this issue Mar 10, 2021
Garderoben pushed a commit that referenced this issue Mar 15, 2021
* added interpolation mechanism

* commit to continue work on laptop

* Cache bust 5.0.5

* first interpolation implementation

* spliced code up to reuse and add in documentation

* Add support for Required parameter to Select component (#1127)

* Add key to MudDialogInstance (#1137)

* Fix #1103 Arithmetic overflow in chart examples

* Fix #1144 Scroll to top example

* fixed initial Y value

* made splined width 100%

* added menu for all the different interpolation algorithms

* Added some base position/visibility classes.

* big cleanup

* cleanup notinterpolation constructor

* Docs: ScrollToTop removed inline <style> css from examples.

* quickly added latest suggestions

* Removed svg height from base scss.

* add action buttons to pickers

* add clear option to pickers

* renamed curveEnum to Interpolationoptions

* change drawer-content height to 100%

* fix #1153

* fix #994

* Docs: Removed docs specific component.

* Add application insights to website

* Add application insights to website

* fix #1169

* Add test today should be selected

* fix appbar/maincontent css in firefox esr

* Fix flaky async autocomplete test

* Fire onclick event in MudNavLink

* fix spelling error

* minor grammar fixes

* improve title grammar

* include subtitle  and description properties for the header

* add information regarding material design surfaces

* minor grammar and punctuation fixes

* Fixed Typo in FlexPage.razor

* MudSwitch removed Class from the internal switch class, Fix for #1163
HClausing pushed a commit to HClausing/MudBlazor that referenced this issue Mar 23, 2021
HClausing pushed a commit to HClausing/MudBlazor that referenced this issue Mar 23, 2021
* added interpolation mechanism

* commit to continue work on laptop

* Cache bust 5.0.5

* first interpolation implementation

* spliced code up to reuse and add in documentation

* Add support for Required parameter to Select component (MudBlazor#1127)

* Add key to MudDialogInstance (MudBlazor#1137)

* Fix MudBlazor#1103 Arithmetic overflow in chart examples

* Fix MudBlazor#1144 Scroll to top example

* fixed initial Y value

* made splined width 100%

* added menu for all the different interpolation algorithms

* Added some base position/visibility classes.

* big cleanup

* cleanup notinterpolation constructor

* Docs: ScrollToTop removed inline <style> css from examples.

* quickly added latest suggestions

* Removed svg height from base scss.

* add action buttons to pickers

* add clear option to pickers

* renamed curveEnum to Interpolationoptions

* change drawer-content height to 100%

* fix MudBlazor#1153

* fix MudBlazor#994

* Docs: Removed docs specific component.

* Add application insights to website

* Add application insights to website

* fix MudBlazor#1169

* Add test today should be selected

* fix appbar/maincontent css in firefox esr

* Fix flaky async autocomplete test

* Fire onclick event in MudNavLink

* fix spelling error

* minor grammar fixes

* improve title grammar

* include subtitle  and description properties for the header

* add information regarding material design surfaces

* minor grammar and punctuation fixes

* Fixed Typo in FlexPage.razor

* MudSwitch removed Class from the internal switch class, Fix for MudBlazor#1163
HClausing pushed a commit to HClausing/MudBlazor that referenced this issue Apr 5, 2021
* added interpolation mechanism

* commit to continue work on laptop

* Cache bust 5.0.5

* first interpolation implementation

* spliced code up to reuse and add in documentation

* Add support for Required parameter to Select component (MudBlazor#1127)

* Add key to MudDialogInstance (MudBlazor#1137)

* Fix MudBlazor#1103 Arithmetic overflow in chart examples

* Fix MudBlazor#1144 Scroll to top example

* fixed initial Y value

* made splined width 100%

* added menu for all the different interpolation algorithms

* Added some base position/visibility classes.

* big cleanup

* cleanup notinterpolation constructor

* Docs: ScrollToTop removed inline <style> css from examples.

* quickly added latest suggestions

* Removed svg height from base scss.

* add action buttons to pickers

* add clear option to pickers

* renamed curveEnum to Interpolationoptions

* change drawer-content height to 100%

* fix MudBlazor#1153

* fix MudBlazor#994

* Docs: Removed docs specific component.

* Add application insights to website

* Add application insights to website

* fix MudBlazor#1169

* Add test today should be selected

* fix appbar/maincontent css in firefox esr

* Fix flaky async autocomplete test

* Fire onclick event in MudNavLink

* fix spelling error

* minor grammar fixes

* improve title grammar

* include subtitle  and description properties for the header

* add information regarding material design surfaces

* minor grammar and punctuation fixes

* Fixed Typo in FlexPage.razor

* MudSwitch removed Class from the internal switch class, Fix for MudBlazor#1163
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something does not work as intended/expected fixed Fix has been merged
Projects
None yet
Development

No branches or pull requests

2 participants