Skip to content

Commit 51e9e9d

Browse files
authored
CmdPal: Update the alias tags, before triggering the UI to change (#38172)
Yea, this one hurts. Pretty obviously, we're sending the PropChanged _before_ the `Tags` are actually updated. Noticed on 0.0.16+ builds
1 parent 5157ffc commit 51e9e9d

File tree

1 file changed

+1
-2
lines changed

1 file changed

+1
-2
lines changed

src/modules/cmdpal/Microsoft.CmdPal.UI.ViewModels/TopLevelViewModel.cs

+1-2
Original file line numberDiff line numberDiff line change
@@ -207,12 +207,11 @@ private void UpdateTags()
207207
tags.Add(new Tag() { Text = Alias.SearchPrefix });
208208
}
209209

210-
PropChanged?.Invoke(this, new PropChangedEventArgs(nameof(Tags)));
211-
212210
DoOnUiThread(
213211
() =>
214212
{
215213
ListHelpers.InPlaceUpdateList(Tags, tags);
214+
PropChanged?.Invoke(this, new PropChangedEventArgs(nameof(Tags)));
216215
});
217216
}
218217

0 commit comments

Comments
 (0)