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

Show file tree
Hide file tree
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
Next Next commit
Revert sample project commit; now replaced by UI test
  • Loading branch information
@nicolaihenriksen
nicolaihenriksen committed Feb 27, 2023
commit 648ca20317e2ede22e9a485f67c817591f14ba22
4 changes: 1 addition & 3 deletions Directory.packages.props
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -26,7 +26,5 @@
<PackageVersion Include="xunit" Version="2.4.2" />
<PackageVersion Include="xunit.runner.visualstudio" Version="2.4.5" />
<PackageVersion Include="Xunit.StaFact" Version="1.1.11" />
<PackageVersion Include="Microsoft.Extensions.DependencyInjection" Version="7.0.0" />
<PackageVersion Include="Microsoft.Extensions.Hosting" Version="7.0.1" />
</ItemGroup>
</Project>
</Project>
13 changes: 0 additions & 13 deletions MDIX-Testing/App.xaml
View file
Open in desktop

This file was deleted.

42 changes: 0 additions & 42 deletions MDIX-Testing/App.xaml.cs
View file
Open in desktop

This file was deleted.

10 changes: 0 additions & 10 deletions MDIX-Testing/AssemblyInfo.cs
View file
Open in desktop

This file was deleted.

22 changes: 0 additions & 22 deletions MDIX-Testing/Dialog.xaml
View file
Open in desktop

This file was deleted.

11 changes: 0 additions & 11 deletions MDIX-Testing/Dialog.xaml.cs
View file
Open in desktop

This file was deleted.

13 changes: 0 additions & 13 deletions MDIX-Testing/InfoView.xaml
View file
Open in desktop

This file was deleted.

26 changes: 0 additions & 26 deletions MDIX-Testing/InfoView.xaml.cs
View file
Open in desktop

This file was deleted.

22 changes: 0 additions & 22 deletions MDIX-Testing/MDIX-Testing.csproj
View file
Open in desktop

This file was deleted.

74 changes: 0 additions & 74 deletions MDIX-Testing/MainWindow.xaml
View file
Open in desktop

This file was deleted.

9 changes: 0 additions & 9 deletions MDIX-Testing/MainWindow.xaml.cs
View file
Open in desktop

This file was deleted.

36 changes: 0 additions & 36 deletions MDIX-Testing/MainWindowViewModel.cs
View file
Open in desktop

This file was deleted.

19 changes: 0 additions & 19 deletions MaterialDesignToolkit.Full.sln
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -63,8 +63,6 @@ Project("{2150E333-8FDC-42A3-9474-1A3956D46DE8}") = "Build", "Build", "{18401177
EndProject
Project("{9A19103F-16F7-4668-BE54-9A1E7A4F7556}") = "MaterialDesign3Demo", "MaterialDesign3.Demo.Wpf\MaterialDesign3Demo.csproj", "{98627CBE-F009-482E-97E9-C69C7135E91F}"
EndProject
Project("{9A19103F-16F7-4668-BE54-9A1E7A4F7556}") = "MDIX-Testing", "MDIX-Testing\MDIX-Testing.csproj", "{00EE118A-86D5-434B-AE42-7DEEE720CF8C}"
EndProject
Global
GlobalSection(SolutionConfigurationPlatforms) = preSolution
Debug|Any CPU = Debug|Any CPU
Expand Down Expand Up @@ -237,22 +235,6 @@ Global
{98627CBE-F009-482E-97E9-C69C7135E91F}.Release|x64.Build.0 = Release|Any CPU
{98627CBE-F009-482E-97E9-C69C7135E91F}.Release|x86.ActiveCfg = Release|Any CPU
{98627CBE-F009-482E-97E9-C69C7135E91F}.Release|x86.Build.0 = Release|Any CPU
{00EE118A-86D5-434B-AE42-7DEEE720CF8C}.Debug|Any CPU.ActiveCfg = Debug|Any CPU
{00EE118A-86D5-434B-AE42-7DEEE720CF8C}.Debug|Any CPU.Build.0 = Debug|Any CPU
{00EE118A-86D5-434B-AE42-7DEEE720CF8C}.Debug|ARM.ActiveCfg = Debug|Any CPU
{00EE118A-86D5-434B-AE42-7DEEE720CF8C}.Debug|ARM.Build.0 = Debug|Any CPU
{00EE118A-86D5-434B-AE42-7DEEE720CF8C}.Debug|x64.ActiveCfg = Debug|Any CPU
{00EE118A-86D5-434B-AE42-7DEEE720CF8C}.Debug|x64.Build.0 = Debug|Any CPU
{00EE118A-86D5-434B-AE42-7DEEE720CF8C}.Debug|x86.ActiveCfg = Debug|Any CPU
{00EE118A-86D5-434B-AE42-7DEEE720CF8C}.Debug|x86.Build.0 = Debug|Any CPU
{00EE118A-86D5-434B-AE42-7DEEE720CF8C}.Release|Any CPU.ActiveCfg = Release|Any CPU
{00EE118A-86D5-434B-AE42-7DEEE720CF8C}.Release|Any CPU.Build.0 = Release|Any CPU
{00EE118A-86D5-434B-AE42-7DEEE720CF8C}.Release|ARM.ActiveCfg = Release|Any CPU
{00EE118A-86D5-434B-AE42-7DEEE720CF8C}.Release|ARM.Build.0 = Release|Any CPU
{00EE118A-86D5-434B-AE42-7DEEE720CF8C}.Release|x64.ActiveCfg = Release|Any CPU
{00EE118A-86D5-434B-AE42-7DEEE720CF8C}.Release|x64.Build.0 = Release|Any CPU
{00EE118A-86D5-434B-AE42-7DEEE720CF8C}.Release|x86.ActiveCfg = Release|Any CPU
{00EE118A-86D5-434B-AE42-7DEEE720CF8C}.Release|x86.Build.0 = Release|Any CPU
EndGlobalSection
GlobalSection(SolutionProperties) = preSolution
HideSolutionNode = FALSE
Expand All @@ -263,7 +245,6 @@ Global
{59CE50AC-D176-4CC0-B465-26F66054A15E} = {9E303A4A-3712-44B9-91EE-830FDC087795}
{18401177-A30E-4330-8F6B-101DF876CE99} = {55087897-5F09-45CA-8E12-12B36B45F262}
{98627CBE-F009-482E-97E9-C69C7135E91F} = {D34BE232-DE51-43C1-ABDC-B69003BB50FF}
{00EE118A-86D5-434B-AE42-7DEEE720CF8C} = {D34BE232-DE51-43C1-ABDC-B69003BB50FF}
EndGlobalSection
GlobalSection(ExtensibilityGlobals) = postSolution
SolutionGuid = {730B2F9E-74AE-46CE-9E61-89AA5C6D5DD3}
Expand Down