Skip to content

Add WinUI layout skill#93

Open
wangdiandao wants to merge 3 commits into
microsoft:stagingfrom
wangdiandao:add-winui-layout-design
Open

Add WinUI layout skill#93
wangdiandao wants to merge 3 commits into
microsoft:stagingfrom
wangdiandao:add-winui-layout-design

Conversation

@wangdiandao
Copy link
Copy Markdown

@wangdiandao wangdiandao commented May 20, 2026

## Summary

  • Add the winui-layout skill for WinUI page layout and responsive design guidance.
  • Add Microsoft Learn layout guidance references for app silhouettes, breakpoints, title bars, content spacing, layout panels, and initial window sizing.
  • Route deeper layout work from winui-design to winui-layout and update README, changelog, issue template, and PR review wording.

Validation

  • Equivalent quick_validate.py frontmatter/name/description checks for plugins\winui\skills\winui-layout: passed.
  • Repo frontmatter validation equivalent for all plugins\winui\skills\**\SKILL.md: passed.
  • git diff --check origin/staging..HEAD: passed.
  • Stale wording scan for eight skills, 8 shipped skills, 8 existing ones, and winui-layout-design: no matches.
  • Version manifest diff check for plugins/winui/plugin.json, .github/plugin/marketplace.json, and .claude-plugin/marketplace.json: no changes.

Notes

  • No C# tools were changed, so scripts/build-tools.ps1 was not run.
  • Version fields and tool payloads were intentionally left unchanged.

@wangdiandao wangdiandao changed the title [codex] Add WinUI layout skill Add WinUI layout skill May 20, 2026
@wangdiandao
Copy link
Copy Markdown
Author

@microsoft-github-policy-service agree


### 5. Integrate the title bar

Use the platform title bar model unless there is a specific reason to customize it.
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Platform == win32 titlebar?

int he new templates we are moving towards the WinUI TitleBar control, I'd suggest we recommend

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes,the platform == titlebar in winui3.

@@ -0,0 +1,116 @@
---
name: winui-layout
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I wonder if this should be part of the design skill vs creating a new one? @nmetulev what are best practices for something like this?

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Comment on lines +72 to +77

- 8 epx: closely related controls or a button and flyout.
- 12 epx: control-to-label or compact content-region spacing.
- 16 epx: text inside a surface edge, or a control next to an expander action.
- 48 epx: nested expander content indentation.
- 56 epx: large media or highly cohesive content margins, not dense tools.
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this is already in the design skill?

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes,but i think it's better to create a new skill.

@wangdiandao wangdiandao marked this pull request as ready for review May 23, 2026 06:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants