When you say "invoke some code that closes the view" you mean destroying the IView, or just taking the TextBlock out of the UI tree?
Anyway, I'm not able to reproduce the issue in any of those cases. I am using the following xaml:

<UserControl x:Class="Testing.TestInlines"

xmlns="http://schemas.microsoft.com/winfx/2006/xaml/presentation"

xmlns:x="http://schemas.microsoft.com/winfx/2006/xaml">

<Grid>

<TextBlock x:Name="txt" HorizontalAlignment="Center" VerticalAlignment="Center" FontSize="50"/>

<Button x:Name="btn" Content="Button" HorizontalAlignment="Center" VerticalAlignment="Bottom" Margin="20"/>

</Grid>

</UserControl>

struct TestInlines: public UserControl

{

TestInlines()

{

InitializeComponent();

    Button* btn = FindName<Button>("btn");
    btn->Click() += [this](BaseComponent*, const RoutedEventArgs&)
    {
        TextBlock* txt = FindName<TextBlock>("txt");
        InlineCollection& inlines = *txt->GetInlines();
        Ptr<Run> run = Noesis::MakePtr<Noesis::Run>();
        run->SetText("Hello World!");
        run->Unloaded() += [](BaseComponent*, const RoutedEventArgs&) { NS_LOG_INFO("Run Unloaded"); };
        inlines.Add(run);
    };
}

void InitializeComponent() { GUI::LoadComponent(this, "InlinesUnloaded.xaml"); }

NS_IMPLEMENT_INLINE_REFLECTION_(TestInlines, UserControl, "Testing.TestInlines")
};

Just destroying the UI element and thus taking it out of the UI tree, not destroying the top-level IView. I'll see if I can do a repro you can easily test.

I do agree that your provided code should repro the issue based on my conclusions and what I wrote in the ticket, so I'll have to take a closer look at this.

Did you find a way to reproduce this behavior?

Hi, sorry that it took so long for me to get back to you. I have now managed to reproduce the issue in a more isolated manner. Try this code and see if it reproduces it for you:

// .h
struct TestInlines: public UserControl
{
TestInlines();
static void OnRegisterUnloadedEventsChanged( Noesis::DependencyObject pSender, const Noesis::DependencyPropertyChangedEventArgs& EventArgs );
static const Noesis::DependencyProperty _pRegisterUnloadedEventsProperty;
NS_DECLARE_REFLECTION( TestInlines, UserControl )
};

// .cpp
TestInlines::TestInlines()
{
Noesis::GUI::LoadComponent( this, "InlinesUnloaded.xaml" );
}

void TestInlines::OnRegisterUnloadedEventsChanged( Noesis::DependencyObject pSender, const Noesis::DependencyPropertyChangedEventArgs& EventArgs )
{
Noesis::TextBlock txt = Noesis::DynamicCast<Noesis::TextBlock*>( pSender );
if ( !txt )
{
return;
}

InlineCollection& inlines = *txt->GetInlines();
Ptr<Run> run = Noesis::MakePtr<Noesis::Run>();
run->SetText("Hello World!");
run->Unloaded() += [](BaseComponent*, const RoutedEventArgs&) { NS_LOG_INFO("Run Unloaded"); };
inlines.Add(run);

}

NS_IMPLEMENT_REFLECTION( CFleetOrderControl, "Testing.TestInlines" )
{
Noesis::UIElementData* pData = NsMeta<Noesis::UIElementData>( Noesis::TypeOf<SelfClass>() );
pData->RegisterProperty<bool>( MessageBoxTextProperty, "RegisterUnloadedEvents", Noesis::PropertyMetadata::Create( false, &OnRegisterUnloadedEventsChanged ) );
}

The dependency property is not the main issue here, I just added it for testing purposes.

This XAML works fine:

<UserControl x:Class="Testing.TestInlines"
xmlns="http://schemas.microsoft.com/winfx/2006/xaml/presentation"
xmlns:x="http://schemas.microsoft.com/winfx/2006/xaml"
xmlns:test="clr-namespace:Testing">
<Grid>
<TextBlock x:Name="txt" HorizontalAlignment="Center" VerticalAlignment="Center" FontSize="50" test:TestInlines.RegisterUnloadedEvents="True" />
</Grid>
</UserControl>

While this does not.
<UserControl x:Class="Testing.TestInlines"
xmlns="http://schemas.microsoft.com/winfx/2006/xaml/presentation"
xmlns:x="http://schemas.microsoft.com/winfx/2006/xaml"
xmlns:test="clr-namespace:Testing">
<Grid>
<Control>
<Control.Template>
<ControlTemplate>
<TextBlock x:Name="txt" HorizontalAlignment="Center" VerticalAlignment="Center" FontSize="50" test:TestInlines.RegisterUnloadedEvents="True" />
</ControlTemplate>
</Control.Template>
</Control>
</Grid>
</UserControl>

So something seems to go wrong when the text block is inside of a control template. It doesn't have to be the direct child of the control template, but it seems like it needs to be directly in the control template's scope, and not eg. be wrapper in a user control that is inside of the control template.

Hello, just wanted to check, has there been another attempt at reproducing this issue?

Here is a patch that fixes the issue with inlines not being Loaded/Unloaded when created inside a template (the changes are on TextBlock::OnPostInit() method).

Index: TextBlock.cpp

--- TextBlock.cpp (revision 13486)
+++ TextBlock.cpp (working copy)
@@ -498,6 +498,18 @@
if (mInlines != 0)
{
ConnectInlines();
+

• if (IsConnectedToView())
• {
• IView* view = GetView();
• int numInlines = mInlines->Count();
• for (int i = 0; i < numInlines; ++i)
• {
• Inline* inline_ = mInlines->Get(i);
• inline_->OnConnectToView(view);
• inline_->OnConnectToViewChildren();
• }

• }
  }

  mTextFlags |= TextFlags_InitializeFinished;
  

If you are not building Noesis from code, I can generate a patched Noesis library for you to try, which version would you need?

Thank you, I will try this out today! We are not building Noesis from source in our usual build setup, but I think I can apply the patch and build Noesis locally.

Nice, it does appear to fix this issue indeed, thanks! :) I'm supposing this will also get into Noesis 3.2.4?

We also have some different issues related to inlines, I was hoping that this fix would fix that too, but it seems that's not the case unfortunately. We're getting a heap corruption when calling Clear() on a Noesis::InlineCollection in some cases (it's related to bindings that we are doing in those inlines). But we'll continue investigating that a bit separately and see if that is also a Noesis bug or if we're doing something wrong with it.

Yes, this will be included in the next 3.2.4 version.

Please open a new ticket regarding the heap corruption deleting inlines if you find it is a Noesis issue, thanks!