style: tree-wide dotnet format; kanban closeout; visual layout demo#8
Merged
Conversation
Formatting-only commit (no logic changes): normalizes the ported code to the .editorconfig standard (2-space indentation, expression spacing). Suite verified unchanged: 1335 passed, 0 failed, 3 skipped. Part of kanban task 139 (restore style enforcement). Co-Authored-By: Claude Fable 5 <[email protected]>
… re-enablement state Targeted dotnet format style pass (readonly members, pattern simplifications in 4 files) and task 139 notes: ~120 gate-on errors remain, dominated by the IDE1006 underscore-prefix field naming question that needs a convention decision. Co-Authored-By: Claude Fable 5 <[email protected]>
Move 101 (yoga-csharp-port epic), 123 (algorithm-helpers), 124 (absolute-layout), 125 (calculate-layout epic), and 138 (integration-testing) to done with results. The port is complete and conformance-verified (1335/0/3, merged via PR #7). Task 139 (restore style enforcement / field naming) stays in to-do; remaining low-priority unit tests and benchmarks are noted in 138's results for future pickup. Co-Authored-By: Claude Fable 5 <[email protected]>
samples/layout-demo/layout-demo.cs renders six representative layouts (grow ratios, wrap+gap, justify/align, nesting, absolute insets, RTL) two ways: ASCII boxes in the terminal, and with --html a page that draws the engine's computed coordinates beside the browser's native CSS flexbox for identical styles - a direct visual correctness check. Co-Authored-By: Claude Fable 5 <[email protected]>
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
1 participant
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
dotnet format(formatting-only commit, 88 files): normalizes the ported code to the .editorconfig 2-space standard, eliminating ~15.5k IDE0055 diagnostics. Test suite verified unchanged before and after: 1335 passed, 0 failed, 3 skipped._camelCasefield naming question (deferred by decision — naming to be fixed later).samples/layout-demo/layout-demo.csrenders six representative layouts as ASCII boxes in the terminal, and with--htmlemits a page drawing the engine's computed coordinates beside the browser's native CSS flexbox for identical styles.Verification
dotnet testafter every commit: 1335 passed / 0 failed / 3 skipped.🤖 Generated with Claude Code