Skip to content

DYN-7099: notification focus fix #16321

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
zeusongit merged 7 commits into from
Jul 2, 2025
Merged

DYN-7099: notification focus fix #16321

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
7 commits
Select commit Hold shift + click to select a range
5da184e
notification focus fix
dnenov Jun 24, 2025
ab7c5bf
fix failing smoke test
dnenov Jun 25, 2025
3f109a6
test if changes are responsible for smoke test failure
dnenov Jun 25, 2025
92f7c94
sync call focus
dnenov Jun 25, 2025
1332b47
clean-up
dnenov Jun 25, 2025
551b3a3
test fail fix
dnenov Jun 25, 2025
e5e1e64
capture errors for native calls
dnenov Jun 30, 2025
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
107 changes: 55 additions & 52 deletions src/Notifications/View/NotificationUI.xaml
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,79 +1,82 @@
<Popup x:Class="Dynamo.Notifications.View.NotificationUI"
xmlns="http://schemas.microsoft.com/winfx/2006/xaml/presentation"
xmlns:x="http://schemas.microsoft.com/winfx/2006/xaml"
xmlns:mc="http://schemas.openxmlformats.org/markup-compatibility/2006"
xmlns:d="http://schemas.microsoft.com/expression/blend/2008"
xmlns:ui="clr-namespace:Dynamo.UI;assembly=DynamoCoreWpf"
xmlns:controls="clr-namespace:Dynamo.Controls;assembly=DynamoCoreWpf"
xmlns:p="clr-namespace:Dynamo.Wpf.Properties;assembly=DynamoCoreWpf"
xmlns:fa="http://schemas.fontawesome.io/icons/"
xmlns:wv2="clr-namespace:Dynamo.Wpf.Utilities;assembly=DynamoCoreWpf"
mc:Ignorable="d"
AllowsTransparency="True"
StaysOpen="False"
Opacity="0.5"
Width="340"
Height="598">
<Popup
x:Class="Dynamo.Notifications.View.NotificationUI"
xmlns="http://schemas.microsoft.com/winfx/2006/xaml/presentation"
xmlns:x="http://schemas.microsoft.com/winfx/2006/xaml"
xmlns:controls="clr-namespace:Dynamo.Controls;assembly=DynamoCoreWpf"
xmlns:d="http://schemas.microsoft.com/expression/blend/2008"
xmlns:fa="http://schemas.fontawesome.io/icons/"
xmlns:mc="http://schemas.openxmlformats.org/markup-compatibility/2006"
xmlns:p="clr-namespace:Dynamo.Wpf.Properties;assembly=DynamoCoreWpf"
xmlns:ui="clr-namespace:Dynamo.UI;assembly=DynamoCoreWpf"
xmlns:wv2="clr-namespace:Dynamo.Wpf.Utilities;assembly=DynamoCoreWpf"
Width="340"
Height="598"
AllowsTransparency="True"
Opacity="0.5"
Opened="NotificationUI_Opened"
StaysOpen="False"
mc:Ignorable="d">
<Popup.Resources>
<ResourceDictionary>
<ResourceDictionary.MergedDictionaries>
<ui:SharedResourceDictionary Source="{x:Static ui:SharedDictionaryManager.DynamoConvertersDictionaryUri}" />
<ui:SharedResourceDictionary Source="{x:Static ui:SharedDictionaryManager.DynamoModernDictionaryUri}" />
</ResourceDictionary.MergedDictionaries>
<BooleanToVisibilityConverter x:Key="BooleanToVisibilityConverter"/>
<controls:PointsToPathConverter x:Key="PointsToPathConverter"/>
<BooleanToVisibilityConverter x:Key="BooleanToVisibilityConverter" />
<controls:PointsToPathConverter x:Key="PointsToPathConverter" />
</ResourceDictionary>
</Popup.Resources>
<Canvas Background="Transparent"
Name="RootLayout" >
<Path x:Name="PopupPathRectangle"
Style="{StaticResource PoupPathRectangleStyle}">
<Canvas Name="RootLayout" Background="Transparent">
<Path x:Name="PopupPathRectangle" Style="{StaticResource PoupPathRectangleStyle}">
<Path.Data>
<RectangleGeometry x:Name="BackgroundRectangle">
</RectangleGeometry>
<RectangleGeometry x:Name="BackgroundRectangle" />
</Path.Data>
<!--This effect will show a 4px shadow of 20% of tranparency with a angle of 135 grades-->
<!-- This effect will show a 4px shadow of 20% of tranparency with a angle of 135 grades -->
<Path.Effect>
<DropShadowEffect Opacity="0.2"
Color="Black"
ShadowDepth="4"
BlurRadius="4"
Direction="135"/>
<DropShadowEffect
BlurRadius="4"
Direction="135"
Opacity="0.2"
ShadowDepth="4"
Color="Black" />
</Path.Effect>
</Path>

<Path x:Name="PopupPathRectangleShadow"
Style="{StaticResource PoupPathRectangleStyle}">
<Path x:Name="PopupPathRectangleShadow" Style="{StaticResource PoupPathRectangleStyle}">
<Path.Data>
<RectangleGeometry Rect="{Binding ElementName=BackgroundRectangle, Path=Rect}">
</RectangleGeometry>
<RectangleGeometry Rect="{Binding ElementName=BackgroundRectangle, Path=Rect}" />
</Path.Data>
<!--This effect will show a 4px shadow of 20% of tranparency with a angle of 315 grades-->
<!-- This effect will show a 4px shadow of 20% of tranparency with a angle of 315 grades -->
<Path.Effect>
<DropShadowEffect Opacity="0.2"
Color="Black"
ShadowDepth="4"
BlurRadius="4"
Direction="315"/>
<DropShadowEffect
BlurRadius="4"
Direction="315"
Opacity="0.2"
ShadowDepth="4"
Color="Black" />
</Path.Effect>
</Path>

<!--This Path will draw on the Canvas the pointer (a triangle)-->
<Path x:Name="TooltipPointer"
Data="{Binding Path=TooltipPointerPoints, Converter={StaticResource PointsToPathConverter}}"
Style="{StaticResource PoupPathPointerStyle}">
<!-- This Path will draw on the Canvas the pointer (a triangle) -->
<Path
x:Name="TooltipPointer"
Data="{Binding Path=TooltipPointerPoints, Converter={StaticResource PointsToPathConverter}}"
Style="{StaticResource PoupPathPointerStyle}">
<Path.Effect>
<DropShadowEffect Opacity="0.2"
Color="Black"
ShadowDepth="4"
BlurRadius="4"
Direction="{Binding Path=ShadowTooltipDirection}"/>
<DropShadowEffect
BlurRadius="4"
Direction="{Binding Path=ShadowTooltipDirection}"
Opacity="0.2"
ShadowDepth="4"
Color="Black" />
</Path.Effect>
</Path>

<Grid x:Name="mainPopupGrid" Background="White"
Width="{Binding PopupRectangleWidth}">
<wv2:DynamoWebView2 Name="webView" ></wv2:DynamoWebView2>
<Grid
x:Name="mainPopupGrid"
Width="{Binding PopupRectangleWidth}"
Background="White">
<wv2:DynamoWebView2 Name="webView" />
</Grid>
</Canvas>
</Popup>
42 changes: 42 additions & 0 deletions src/Notifications/View/NotificationUI.xaml.cs
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,6 +1,10 @@
using System;
using System.Reflection;
using System.Runtime.InteropServices;
using System.Windows;
using System.Windows.Controls.Primitives;
using System.Windows.Interop;
using System.Windows.Threading;

namespace Dynamo.Notifications.View
{
Expand Down Expand Up @@ -40,6 +44,8 @@ public NotificationUI()
mainPopupGrid.Width = notificationsUIViewModel.PopupRectangleWidth;
mainPopupGrid.Height = notificationsUIViewModel.PopupRectangleHeight;
mainPopupGrid.Margin = new Thickness(notificationsUIViewModel.PopupBordersOffSet, notificationsUIViewModel.PopupBordersOffSet, 0, 0);


}

internal void UpdatePopupLocation()
Expand All @@ -60,5 +66,41 @@ internal void UpdatePopupSize()

this.Height = notificationsUIViewModel.PopupRectangleHeight + notificationsUIViewModel.PopupBordersOffSet + 10;
}

// Wait until the Popup is opened to set focus back to the main window.
// This is necessary because Popup opens asynchronously, and WebView2 (HWND-based)
// can steal focus. We explicitly reclaim native focus to ensure correct interaction
// with the main window (e.g., close button, click handling).
private void NotificationUI_Opened(object sender, EventArgs e)
{
// Ensure the application and main window are available and fully loaded.
ForceMainWindowFocus();
}

private void ForceMainWindowFocus()
{
if (Application.Current?.MainWindow == null)
return;

if (!ReleaseCapture())
{
int errorCode = Marshal.GetLastWin32Error();
Console.WriteLine($"ReleaseCapture failed with error code: {errorCode}");
}

if (!SetForegroundWindow(new WindowInteropHelper(Application.Current.MainWindow).Handle))
{
int errorCode = Marshal.GetLastWin32Error();
Console.WriteLine($"SetForegroundWindow failed with error code: {errorCode}");
}
}

[DllImport("user32.dll", SetLastError = true)]
[return: MarshalAs(UnmanagedType.Bool)]
private static extern bool ReleaseCapture();

[DllImport("user32.dll", SetLastError = true)]
[return: MarshalAs(UnmanagedType.Bool)]
private static extern bool SetForegroundWindow(IntPtr hWnd);
}
}
Loading