fix(ui): unify titlebar centerline vertical alignment and eliminate redundant brand container (v1.0.14)

2026-09-04 23:58:41 +09:00 · 2026-09-04 23:58:41 +09:00 · 1f794e9e08
commit 1f794e9e08
parent 8eb48ed854
14 changed files with 2323 additions and 660 deletions
--- a/src/Everything2Everything.Tests/UnifiedTitleBarTests.cs
+++ b/src/Everything2Everything.Tests/UnifiedTitleBarTests.cs
@ -0,0 +1,330 @@
+using System.IO;
+using System.Linq;
+using System.Threading;
+using System.Windows;
+using System.Windows.Controls;
+using System.Xml.Linq;
+using Everything2Everything.App.Views;
+using Everything2Everything.Core;
+using Xunit;
+
+namespace Everything2Everything.Tests;
+
+public class UnifiedTitleBarTests
+{
+    private static readonly string SolutionRoot = FindSolutionRoot();
+    private static readonly string MainWindowXamlPath = Path.Combine(SolutionRoot, "src", "Everything2Everything.App", "Views", "MainWindow.xaml");
+
+    private static string FindSolutionRoot()
+    {
+        var dir = Directory.GetCurrentDirectory();
+        while (dir != null && !File.Exists(Path.Combine(dir, "Everything2Everything.slnx")))
+        {
+            dir = Directory.GetParent(dir)?.FullName;
+        }
+        return dir ?? throw new DirectoryNotFoundException("솔루션 루트를 찾을 수 없습니다.");
+    }
+
+    private sealed class FakeSettingsStore : ISettingsStore
+    {
+        private readonly Dictionary<string, string> _d = new();
+        public string? Get(string key) => _d.TryGetValue(key, out var v) ? v : null;
+        public void Set(string key, string value) => _d[key] = value;
+        public void Remove(string key) => _d.Remove(key);
+        public bool Contains(string key) => _d.ContainsKey(key);
+    }
+
+    [Fact]
+    public void MainWindow_MustNotHave_Isolated32PxTitleBarRow()
+    {
+        // 32px짜리 고립된 빈 타이틀바 행이 존재하지 않아야 하며, 
+        // 타이틀바와 윈도우 조작 버튼(최소/최대/닫기)이 메인 헤더와 하나의 통합 바로 통합되어야 한다.
+        var doc = XDocument.Parse(File.ReadAllText(MainWindowXamlPath));
+
+        var rootGrid = doc.Root?.Elements().FirstOrDefault(e => e.Name.LocalName == "Grid");
+        Assert.NotNull(rootGrid);
+
+        var rowDefs = rootGrid.Element(rootGrid.Name.Namespace + "Grid.RowDefinitions")?.Elements().ToList();
+        Assert.NotNull(rowDefs);
+
+        var has32PxRow = rowDefs.Any(r => r.Attribute("Height")?.Value == "32");
+        Assert.False(has32PxRow, "최상위 Grid에 32px 분리된 타이틀바 행이 여전히 존재합니다. 헤더와 통합된 단일 바로 구성되어야 합니다.");
+    }
+
+    [Fact]
+    public void MainWindow_TitleBar_MustBeIntegratedWithTopHeader()
+    {
+        // ui:TitleBar 가 분리된 빈 줄이 아니라 통합 헤더 영역(Height >= 48) 내에 배치되어야 한다.
+        var doc = XDocument.Parse(File.ReadAllText(MainWindowXamlPath));
+
+        var titleBar = doc.Descendants().FirstOrDefault(e => e.Name.LocalName == "TitleBar");
+        Assert.NotNull(titleBar);
+
+        var gridRow = titleBar.Attribute("Grid.Row")?.Value;
+        
+        var rootGrid = doc.Root?.Elements().FirstOrDefault(e => e.Name.LocalName == "Grid");
+        var rowDefs = rootGrid?.Element(rootGrid.Name.Namespace + "Grid.RowDefinitions")?.Elements().ToList();
+        
+        if (gridRow != null && int.TryParse(gridRow, out int rowIndex) && rowDefs != null && rowIndex < rowDefs.Count)
+        {
+            var rowHeight = rowDefs[rowIndex].Attribute("Height")?.Value;
+            Assert.NotEqual("32", rowHeight);
+        }
+    }
+
+    private static readonly object AppInitLock = new();
+
+    [Fact]
+    public void MainWindow_UnifiedTitleBar_MeasuresAndArrangesCorrectly()
+    {
+        Exception? ex = null;
+        var thread = new Thread(() =>
+        {
+            try
+            {
+                lock (AppInitLock)
+                {
+                    if (Application.Current == null)
+                    {
+                        try { _ = new Application(); } catch (InvalidOperationException) { }
+                    }
+                }
+
+                var engine = Everything2EverythingBootstrap.CreateDefault();
+                var store = new FakeSettingsStore();
+                var window = new MainWindow(engine, store);
+
+                var content = (UIElement)window.Content;
+                content.Measure(new Size(1280, 960));
+                content.Arrange(new Rect(0, 0, 1280, 960));
+
+                Assert.True(content.DesiredSize.Width > 0);
+                Assert.True(content.DesiredSize.Height > 0);
+
+                var titleBar = FindLogicalChild<Wpf.Ui.Controls.TitleBar>(window);
+                Assert.NotNull(titleBar);
+                Assert.True(titleBar.ActualHeight >= 40 || titleBar.DesiredSize.Height >= 40,
+                    $"통합 TitleBar의 높이가 최소 40px 이상이어야 합니다. (실제: {titleBar.DesiredSize.Height})");
+            }
+            catch (Exception e)
+            {
+                ex = e;
+            }
+        });
+        thread.SetApartmentState(ApartmentState.STA);
+        thread.Start();
+        thread.Join();
+
+        if (ex != null)
+        {
+            throw new AggregateException("STA 스레드 실행 중 예외 발생", ex);
+        }
+    }
+
+    [Fact]
+    public void MainWindow_MustNotHave_ClunkyBrandContainer_Or_RedundantTitleText()
+    {
+        // 사용자가 불필요하고 이상하다고 지적한 상단 좌측 320px 컨테이너 및 'FormatShift Utility' 문구가 완전히 제거되었는지 정적 AST 검증
+        var doc = XDocument.Parse(File.ReadAllText(MainWindowXamlPath));
+        var titleBar = doc.Descendants().FirstOrDefault(e => e.Name.LocalName == "TitleBar");
+        Assert.NotNull(titleBar);
+
+        var header = titleBar.Elements().FirstOrDefault(e => e.Name.LocalName == "TitleBar.Header");
+        Assert.NotNull(header);
+
+        var allTextBlocks = header.Descendants().Where(e => e.Name.LocalName == "TextBlock" || e.Name.LocalName == "Run").ToList();
+        var redundantTexts = allTextBlocks.Where(tb => 
+            (tb.Attribute("Text")?.Value?.Contains("FormatShift") ?? false) ||
+            (tb.Attribute("Text")?.Value?.Contains("Utility") ?? false) ||
+            tb.Value.Contains("FormatShift") || tb.Value.Contains("Utility")).ToList();
+
+        Assert.Empty(redundantTexts);
+
+        // 320px 너비의 패널 컨테이너 Border가 존재하지 않아야 함
+        var columnDefs = header.Descendants().Where(e => e.Name.LocalName == "ColumnDefinition").ToList();
+        var has320Col = columnDefs.Any(c => c.Attribute("Width")?.Value == "320");
+        Assert.False(has320Col, "상단 바에 320px짜리 투박한 사이드바 헤더 컨테이너가 남아있습니다. 제거되어야 합니다.");
+    }
+
+    [Fact]
+    public void MainWindow_TopHeader_AllElements_MustSharePreciseVerticalCenterline()
+    {
+        // 상단 바 내 모든 요소(최소/최대/닫기 창 버튼, 탭, 엔진 뱃지, 액션 버튼)가
+        // 시각적으로 들뜨거나 가라앉지 않고 동일한 수직 중심선(Centerline, 오차 2.0px 이내)에 완벽 정렬되어야 한다.
+        RunOnSta(() =>
+        {
+            var engine = Everything2EverythingBootstrap.CreateDefault();
+            var store = new FakeSettingsStore();
+            var window = new MainWindow(engine, store);
+
+            var content = (UIElement)window.Content;
+            content.Measure(new Size(1280, 960));
+            content.Arrange(new Rect(0, 0, 1280, 960));
+
+            var titleBar = (Wpf.Ui.Controls.TitleBar)window.FindName("AppTitleBar");
+            Assert.NotNull(titleBar);
+            titleBar.ApplyTemplate();
+
+            var minBtn = (FrameworkElement?)titleBar.Template.FindName("PART_MinimizeButton", titleBar);
+            Assert.NotNull(minBtn);
+
+            var tabBtn = (FrameworkElement)window.FindName("TabActiveBtn");
+            var badge = (FrameworkElement)window.FindName("EngineTelemetryBadge");
+            var clearBtn = (FrameworkElement)window.FindName("ClearAllButton");
+
+            // TitleBar 좌표계 기준 각 컨트롤의 수직 중심 Y좌표 계산
+            double CenterY(FrameworkElement element)
+            {
+                var pt = element.TransformToAncestor(titleBar).Transform(new Point(0, element.ActualHeight / 2.0));
+                return pt.Y;
+            }
+
+            var minCenterY = CenterY(minBtn);
+            var tabCenterY = CenterY(tabBtn);
+            var badgeCenterY = CenterY(badge);
+            var clearCenterY = CenterY(clearBtn);
+
+            // 오차 2.0px 이내로 모든 요소가 동일한 수직 기준선상에 정렬되어야 함
+            Assert.True(Math.Abs(tabCenterY - minCenterY) <= 2.0,
+                $"최소화 버튼 중심선({minCenterY:F1}px)과 탭 중심선({tabCenterY:F1}px)의 수직 정렬 오차가 너무 큽니다.");
+            Assert.True(Math.Abs(badgeCenterY - minCenterY) <= 2.0,
+                $"최소화 버튼 중심선({minCenterY:F1}px)과 엔진 뱃지 중심선({badgeCenterY:F1}px)의 수직 정렬 오차가 너무 큽니다.");
+            Assert.True(Math.Abs(clearCenterY - minCenterY) <= 2.0,
+                $"최소화 버튼 중심선({minCenterY:F1}px)과 액션 버튼 중심선({clearCenterY:F1}px)의 수직 정렬 오차가 너무 큽니다.");
+
+            try
+            {
+                var rtb = new System.Windows.Media.Imaging.RenderTargetBitmap(1280, 720, 96, 96, System.Windows.Media.PixelFormats.Pbgra32);
+                rtb.Render(content);
+                var enc = new System.Windows.Media.Imaging.PngBitmapEncoder();
+                enc.Frames.Add(System.Windows.Media.Imaging.BitmapFrame.Create(rtb));
+                var outPath = @"C:\Users\encep\.gemini\antigravity\brain\b58fd023-a52b-4f4b-aa23-e6df654aa1fb\.tempmediaStorage\rendered_window.png";
+                using var fs = File.Create(outPath);
+                enc.Save(fs);
+            }
+            catch { }
+        });
+    }
+
+    [Fact]
+    public void MainWindow_HeaderElements_MustBeIntegratedIntoTitleBarHeaderAndTrailingContent_ToPreventNcHitTestBlocking()
+    {
+        // Wpf.Ui.Controls.TitleBar의 HwndSourceHook(WM_NCHITTEST)은
+        // 오직 Header, CenterContent, TrailingContent 내부에 포함된 요소만 HTCLIENT(클릭 허용)로 반환하며,
+        // 그 외의 형제 요소나 TitleBar 외부 겹침 요소는 모두 HTCAPTION(창 이동 드래그)으로 가로채 클릭을 차단한다.
+        // 따라서 상단 탭, 뱃지, 액션 버튼들은 반드시 TitleBar.Header와 TitleBar.TrailingContent 내에 직접 배치되어야 한다.
+        RunOnSta(() =>
+        {
+            var engine = Everything2EverythingBootstrap.CreateDefault();
+            var store = new FakeSettingsStore();
+            var window = new MainWindow(engine, store);
+
+            var content = (UIElement)window.Content;
+            content.Measure(new Size(1280, 960));
+            content.Arrange(new Rect(0, 0, 1280, 960));
+
+            var tb = (Wpf.Ui.Controls.TitleBar)window.FindName("AppTitleBar");
+            Assert.NotNull(tb);
+
+            Assert.NotNull(tb.Header);
+            Assert.NotNull(tb.TrailingContent);
+
+            var tabActive = window.FindName("TabActiveBtn") as UIElement;
+            Assert.NotNull(tabActive);
+
+            var telemetryBadge = window.FindName("EngineTelemetryBadge") as UIElement;
+            Assert.NotNull(telemetryBadge);
+
+            var clearBtn = window.FindName("ClearAllButton") as UIElement;
+            Assert.NotNull(clearBtn);
+
+            // tabActive와 telemetryBadge가 tb.Header의 자식인지 검증
+            var headerElement = tb.Header as DependencyObject;
+            Assert.NotNull(headerElement);
+            Assert.True(IsDescendantOf(tabActive, headerElement), "TabActiveBtn은 반드시 AppTitleBar.Header 내부에 배치되어야 클릭이 차단되지 않습니다.");
+            Assert.True(IsDescendantOf(telemetryBadge, headerElement), "EngineTelemetryBadge는 AppTitleBar.Header 내부에 배치되어야 합니다.");
+
+            // clearBtn이 tb.TrailingContent의 자식인지 검증
+            var trailingElement = tb.TrailingContent as DependencyObject;
+            Assert.NotNull(trailingElement);
+            Assert.True(IsDescendantOf(clearBtn, trailingElement), "ClearAllButton은 반드시 AppTitleBar.TrailingContent 내부에 배치되어야 클릭이 차단되지 않습니다.");
+        });
+    }
+
+    [Fact]
+    public void MainWindow_HeaderLayout_MustBeSpaciousAndNonOverlapping()
+    {
+        // 최소 지원 해상도(1080px)에서도 좌측 탭과 우측 액션 버튼이 충돌하지 않도록,
+        // 우측 액션 툴바(설정, 메뉴, 진단, 로그, 비우기)의 총 가로폭이 260px 이하여야 한다.
+        RunOnSta(() =>
+        {
+            var engine = Everything2EverythingBootstrap.CreateDefault();
+            var store = new FakeSettingsStore();
+            var window = new MainWindow(engine, store);
+
+            var content = (UIElement)window.Content;
+            content.Measure(new Size(1080, 720));
+            content.Arrange(new Rect(0, 0, 1080, 720));
+
+            var tb = (Wpf.Ui.Controls.TitleBar)window.FindName("AppTitleBar");
+            Assert.NotNull(tb);
+
+            var trailing = tb.TrailingContent as FrameworkElement;
+            Assert.NotNull(trailing);
+            Assert.True(trailing.DesiredSize.Width <= 260,
+                $"우측 액션 버튼 그룹의 가로 폭이 너무 넓습니다({trailing.DesiredSize.Width}px). 260px 이하로 컴팩트하게 구성되어야 왼쪽 탭과 겹치지 않습니다.");
+        });
+    }
+
+    private static bool IsDescendantOf(DependencyObject? node, DependencyObject targetAncestor)
+    {
+        while (node != null)
+        {
+            if (node == targetAncestor) return true;
+            node = LogicalTreeHelper.GetParent(node) ?? (node as FrameworkElement)?.Parent;
+        }
+        return false;
+    }
+
+    private static void RunOnSta(Action action)
+    {
+        Exception? ex = null;
+        var thread = new Thread(() =>
+        {
+            try
+            {
+                lock (AppInitLock)
+                {
+                    if (Application.Current == null)
+                    {
+                        try { _ = new Application(); } catch (InvalidOperationException) { }
+                    }
+                }
+                action();
+            }
+            catch (Exception e)
+            {
+                ex = e;
+            }
+        });
+        thread.SetApartmentState(ApartmentState.STA);
+        thread.Start();
+        thread.Join();
+
+        if (ex != null) throw new AggregateException("STA 스레드 실행 중 예외 발생", ex);
+    }
+
+    private static T? FindLogicalChild<T>(DependencyObject parent) where T : DependencyObject
+    {
+        foreach (var child in LogicalTreeHelper.GetChildren(parent))
+        {
+            if (child is T typed) return typed;
+            if (child is DependencyObject dep)
+            {
+                var found = FindLogicalChild<T>(dep);
+                if (found != null) return found;
+            }
+        }
+        return null;
+    }
+}