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

Dialog: reverting regression by #3252. Issue #3701 #3752

Merged
merged 2 commits into from
Jan 18, 2022
Merged

Conversation

henon
Copy link
Collaborator

@henon henon commented Jan 18, 2022

Revert regression by PR #3252
Fixes #3701

The intended improvement by #3252 will have to be implemented differently, I don't know yet what exactly the problem is/was that prompted the code change.

@codecov
Copy link

codecov bot commented Jan 18, 2022

Codecov Report

Merging #3752 (1516a63) into dev (23b9dab) will not change coverage.
The diff coverage is 100.00%.

Impacted file tree graph

@@           Coverage Diff           @@
##              dev    #3752   +/-   ##
=======================================
  Coverage   90.58%   90.58%           
=======================================
  Files         332      332           
  Lines       10509    10509           
=======================================
  Hits         9520     9520           
  Misses        989      989           
Impacted Files Coverage Δ
src/MudBlazor/Components/Dialog/MudDialog.razor.cs 96.36% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 23b9dab...1516a63. Read the comment docs.

@henon henon mentioned this pull request Jan 18, 2022
2 tasks
@henon
Copy link
Collaborator Author

henon commented Jan 18, 2022

@peerpalo, Looks like your test is working now with my little change ;)

@henon henon merged commit 8456bbe into MudBlazor:dev Jan 18, 2022
@henon henon added this to the 6.0.6 milestone Jan 18, 2022
@henon henon added the bug Something does not work as intended/expected label Jan 18, 2022
jammerware pushed a commit to jammerware/MudBlazor that referenced this pull request Sep 20, 2022
…udBlazor#3752)

* Dialog: reverting regression by MudBlazor#3252. Issue MudBlazor#3701

* Dialog test: trying to get it to work on CI
jammerware pushed a commit to jammerware/MudBlazor that referenced this pull request Sep 20, 2022
…udBlazor#3752)

* Dialog: reverting regression by MudBlazor#3252. Issue MudBlazor#3701

* Dialog test: trying to get it to work on CI
jammerware pushed a commit to jammerware/MudBlazor that referenced this pull request Sep 20, 2022
…udBlazor#3752)

* Dialog: reverting regression by MudBlazor#3252. Issue MudBlazor#3701

* Dialog test: trying to get it to work on CI
jammerware pushed a commit to jammerware/MudBlazor that referenced this pull request Sep 20, 2022
…udBlazor#3752)

* Dialog: reverting regression by MudBlazor#3252. Issue MudBlazor#3701

* Dialog test: trying to get it to work on CI
3dots pushed a commit to 3dots/MudBlazor that referenced this pull request Mar 23, 2023
…udBlazor#3752)

* Dialog: reverting regression by MudBlazor#3252. Issue MudBlazor#3701

* Dialog test: trying to get it to work on CI
ferraridavide pushed a commit to ferraridavide/MudBlazor that referenced this pull request May 30, 2023
…udBlazor#3752)

* Dialog: reverting regression by MudBlazor#3252. Issue MudBlazor#3701

* Dialog test: trying to get it to work on CI
ilovepilav pushed a commit to ilovepilav/MudBlazor that referenced this pull request Nov 25, 2023
…udBlazor#3752)

* Dialog: reverting regression by MudBlazor#3252. Issue MudBlazor#3701

* Dialog test: trying to get it to work on CI
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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

inline MudDialog problem
1 participant