UI fixes + headless layout-regression test suite for all section views#939
Open
Zazawowow wants to merge 5 commits into
Open
UI fixes + headless layout-regression test suite for all section views#939Zazawowow wants to merge 5 commits into
Zazawowow wants to merge 5 commits into
Conversation
added 4 commits
July 19, 2026 19:04
- Settings: currency ComboBox row, Export/Show Logs button pair and the Danger Zone subtitle now wrap instead of escaping their cards on phones - FindProjects: search error strip converted from horizontal StackPanel to Grid so TextWrapping actually engages on narrow widths - InvestPage: quick-amount buttons get MinWidth=0 + CharacterEllipsis so long amounts can't push past their UniformGrid cell
- Add breakpoint-width layout audits for Home, FindProjects, ProjectDetail, InvestPage, Portfolio, Funds, Settings, CreateProject (all 6 wizard steps) and EditProfile with worst-case data (25 -> 92 tests, still ~11s, offline) - Add generic detach/re-attach test asserting every responsive section view re-creates its LayoutModeService subscription (section-switch bug class) - Harness: skip Popup (windowless overlay) and ScrollContentPresenter overflow (scrolling is the feature); audit ContentPresenter content instead of the wrapper to ignore UniformGrid 1-2px arrange rounding
dangershony
reviewed
Jul 20, 2026
| // (docs/ai-docs/taproot-arm64-jit-bug.md) persisted key sets where every | ||
| // slot carries the same project identifier. Re-derive from the seed | ||
| // (OS-secured key store, no prompt) and persist the corrected set. | ||
| var distinctIds = keysResult.Value.Keys.Select(k => k.ProjectIdentifier).Distinct().Count(); |
Member
There was a problem hiding this comment.
not clear what's going on here needs further investigation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
src/design/App.Test.Integration/LayoutRegression/): renders section views with real theme + Skia text metrics at breakpoint widths (360/390/767/768/1023/1280) and asserts no overlapping siblings, no children overflowing parent panels, and responsive subscriptions surviving detach/re-attach. 92 tests, ~11s, fully offline. Wired into CI.Test plan
dotnet test src/design/App.Test.Integration/App.Test.Integration.csproj --filter "FullyQualifiedName~LayoutRegression"— 92/92 passing (verified after rebase onto main)