Skip to content

Properly restore focus when TabControl.SelectedIndex is set via DialogHost #3104

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.

Sign up for GitHub

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

Merged
Keboo merged 11 commits into from
Feb 28, 2023
Merged

Properly restore focus when TabControl.SelectedIndex is set via DialogHost #3104

Changes from 1 commit
Commits
Show all changes
11 commits
Select commit Hold shift + click to select a range
c80a75e
Add sample project reproducing the issue
nicolaihenriksen Feb 25, 2023
b5031fb
"Hacky fix" to showcase what the root cause is
nicolaihenriksen Feb 25, 2023
0182649
Remove hacky fix
nicolaihenriksen Feb 27, 2023
6cb4c8b
Add failing UI test
nicolaihenriksen Feb 27, 2023
648ca20
Revert sample project commit; now replaced by UI test
nicolaihenriksen Feb 27, 2023
b39b73a
Add RestoreFocusElement DP to make test run green
nicolaihenriksen Feb 27, 2023
3d5e3eb
Add failing UI test for "disable" scenario
nicolaihenriksen Feb 27, 2023
64764d8
Add DialogHost.IsFocusRestoreDisabled DP to opt. disable focus restore
nicolaihenriksen Feb 27, 2023
adf82e2
Rename DP to DialogHost.IsRestoreFocusDisabled
nicolaihenriksen Feb 28, 2023
bb51d40
Add UI tests for navigation rail
nicolaihenriksen Feb 28, 2023
deeb97a
Region rephrase
nicolaihenriksen Feb 28, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Region rephrase
  • Loading branch information
@nicolaihenriksen
nicolaihenriksen committed Feb 28, 2023
commit deeb97a8380c6ffaa74ae583267332633df6cbc2
2 changes: 1 addition & 1 deletion MaterialDesignThemes.Wpf/DialogHost.cs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -612,7 +612,7 @@ public override void OnApplyTemplate()
base.OnApplyTemplate();
}

#region focus restore properties
#region restore focus properties

public static readonly DependencyProperty RestoreFocusElementProperty = DependencyProperty.RegisterAttached(
"RestoreFocusElement", typeof(IInputElement), typeof(DialogHost), new PropertyMetadata(default(IInputElement)));
Expand Down