Skip to content

Commit c2426c0

Browse files
csharpfritzCopilot
andcommitted
fix: remove leftover LogError debug spam from UiStyle
UiStyle.OnInitialized() was calling LogError for every BackColor set, causing false console errors in Playwright tests. Also mark WingtipToys migration as complete in README. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
1 parent 814beca commit c2426c0

2 files changed

Lines changed: 2 additions & 13 deletions

File tree

docs/migration-tests/README.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,4 +4,4 @@ Benchmarked migrations of real Web Forms applications using the BWFC migration t
44

55
| Run | Source App | Date | Pages | Result |
66
|-----|-----------|------|-------|--------|
7-
| [WingtipToys](wingtiptoys-2026-03-04/report.md) | WingtipToys 2013 | 2026-03-04 | 32 | In progress |
7+
| [WingtipToys](wingtiptoys-2026-03-04/report.md) | WingtipToys 2013 | 2026-03-04 | 32 | ✅ Complete |

src/BlazorWebFormsComponents/UiStyle.cs

Lines changed: 1 addition & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,5 @@
11
using BlazorWebFormsComponents.Enums;
22
using Microsoft.AspNetCore.Components;
3-
using Microsoft.Extensions.Logging;
43
using System.Collections.Generic;
54

65
namespace BlazorWebFormsComponents
@@ -56,18 +55,8 @@ public abstract class UiStyle<TStyle> : ComponentBase, IHasLayoutStyle where TSt
5655
[Parameter]
5756
public Unit Width { get; set; }
5857

59-
[Inject]
60-
public ILoggerFactory LoggerFactory { get; set; }
61-
62-
protected override void OnInitialized()
58+
protected override void OnInitialized()
6359
{
64-
65-
var thisLogger = LoggerFactory.CreateLogger("Style");
66-
if (BackColor != null)
67-
{
68-
thisLogger.LogError($"Backcolor inside the UiStyle is: {BackColor.ToColor().ToString()}");
69-
}
70-
7160
if (theStyle != null)
7261
{
7362

0 commit comments

Comments
 (0)