KofTwentyTwo/AppKit
2
0 Fork 0
Code Issues 22 Pull Requests 4 Actions Packages Projects Releases 1 Wiki Activity

WIP: feat(samples): add repository issue and community Help links #29

Draft
KofTwentyTwo wants to merge 1 commits from feat/help-menu into main
pull from: feat/help-menu
merge into: KofTwentyTwo:main
Conversation 0 Commits 1 Files Changed 8
+141 -3
KofTwentyTwo commented 2026-10-11 21:33:47 +00:00 (Migrated from github.com)
Copy link
Copy source

Both AppKit samples now expose Help → View on GitHub, Submit an issue… and Community support…, matching gclo’s support pattern. Destinations derive from the sample’s own repository; browser-launch failures surface a generic status without document content or paths. Existing About, updates and activity commands remain available.

Adds real native menu acceptance for both WinUI and WPF, checking command availability and labels without opening an external browser. No public AppKit API or package dependency changes.

Validation: locked restore and full Release x64 build pass with zero warnings/errors; 115 unit tests pass with 100% measured core and updates line coverage; formatting and signed/DCO commit hooks pass. Exactf58aac9 CI38176887951 hosted native14/14 passes, zero failures/skips, including both WinUI and WPF menus. Live AppKit Discussions is enabled. Screenshots and current-head maintainer review remain required before merge.

Refs #28. Substantial AI assistance: implementation, tests and review by Codex (GPT-6).

Both AppKit samples now expose Help → View on GitHub, Submit an issue… and Community support…, matching gclo’s support pattern. Destinations derive from the sample’s own repository; browser-launch failures surface a generic status without document content or paths. Existing About, updates and activity commands remain available. Adds real native menu acceptance for both WinUI and WPF, checking command availability and labels without opening an external browser. No public AppKit API or package dependency changes. Validation: locked restore and full Release x64 build pass with zero warnings/errors; 115 unit tests pass with 100% measured core and updates line coverage; formatting and signed/DCO commit hooks pass. Exactf58aac9 CI38176887951 hosted native14/14 passes, zero failures/skips, including both WinUI and WPF menus. Live AppKit Discussions is enabled. Screenshots and current-head maintainer review remain required before merge. Refs #28. Substantial AI assistance: implementation, tests and review by Codex (GPT-6).
You are not authorized to merge this pull request.
This pull request can be merged automatically.
This branch is out-of-date with the base branch
This pull request is marked as a work in progress.
View command line instructions

Checkout

From your project repository, check out a new branch and test the changes.
git fetch -u origin feat/help-menu:feat/help-menu
git checkout feat/help-menu
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
Clear labels
accessibility
Barrier affecting people with disabilities
 bug
Something isn't working
 deferred
Agreed follow-up, unscheduled
 dependencies
Pull requests that update a dependency file
 documentation
Improvements or additions to documentation
 duplicate
This issue or pull request already exists
 enhancement
New feature or request
 github_actions
Pull requests that update GitHub Actions code
 good first issue
Good for newcomers
 help wanted
Extra attention is needed
 invalid
This doesn't seem right
 needs-owner
Blocked on the owner
 priority
Work this first
 quality
Tests, gates, and code quality
 question
Further information is requested
 refactor
Internal restructuring, no behavior change
 release
Release and packaging
 wontfix
This will not be worked on
No labels
Milestone
No items
No Milestone
Projects
Clear projects
No projects
Assignees
Clear assignees
forge-recovery (Forge Recovery) james.maes (James Maes)
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: KofTwentyTwo/AppKit#29