Skip to content

feat(ux): add Apple's 10 Essential Design Principles (WWDC 2017)#351

Open
fazalrshah wants to merge 1 commit into
nextlevelbuilder:mainfrom
fazalrshah:feat/apple-essential-design-principles
Open

feat(ux): add Apple's 10 Essential Design Principles (WWDC 2017)#351
fazalrshah wants to merge 1 commit into
nextlevelbuilder:mainfrom
fazalrshah:feat/apple-essential-design-principles

Conversation

@fazalrshah

Copy link
Copy Markdown

What & why

Adds Apple's ten Essential Design Principles from WWDC 2017 — Session 802 "Essential Design Principles" (Mike Stern) to the UX knowledge base. These are timeless, human-centered product principles that complement the skill's existing tactical rules — they describe the why (the experience you're designing for) where the current rule categories describe the measurable how.

The talk's framing — design to meet four human needs: safety, meaning, achievement, and beauty — is captured as the section intro.

Changes

Data (the core contribution)

  • src/ui-ux-pro-max/data/ux-guidelines.csv+10 rows under a new Apple Design Principles category, each with description / Do / Don't / good + bad examples / severity:
    Wayfinding · Feedback · Visibility · Consistency · Mental Models · Proximity & Grouping · Mapping · Affordances · Progressive Disclosure · Symmetry & Balance
  • These are picked up by search.py and surface via --domain ux and the --design-system flow.

Guidance

  • src/ui-ux-pro-max/templates/base/quick-reference.md + .claude/skills/ui-ux-pro-max/SKILL.md — a new "Foundational Design Principles" section placed just above the Rule Categories table, so the principles are read at the concept stage.

Housekeeping

  • Bumped the advertised guideline count 99 → 109 in skill.json, README.md, and the platform template descriptions.
  • Synced src/cli/assets/ per the contributor guide in the README.

Verification

  • CSV parses cleanly: 109 rows, no gaps, CRLF preserved (append-only diff).
  • All changed JSON validated.
  • search.py "apple design principles" --domain ux returns the new rows.
  • cli/assets kept byte-in-sync with src/ for the touched files.

Notes

  • CSV append preserves the file's existing CRLF line endings (the diff is purely additive).
  • Principle definitions are sourced directly from the session; phrasing condensed for the dataset format.

Add the ten foundational, human-centered design principles from Apple's
WWDC 2017 "Essential Design Principles" (session 802) to the UX knowledge
base, so they are both searchable data and concept-stage guidance.

- data/ux-guidelines.csv: +10 rows under category "Apple Design Principles"
  (Wayfinding, Feedback, Visibility, Consistency, Mental Models,
  Proximity & Grouping, Mapping, Affordances, Progressive Disclosure,
  Symmetry & Balance) — each with description / do / don't / examples /
  severity. Surfaced via `--domain ux` and `--design-system`.
- templates/base/quick-reference.md + .claude SKILL.md: add a "Foundational
  Design Principles" section framing the *why* behind the tactical rule
  categories (the talk's safety / meaning / achievement / beauty goal).
- Bump advertised guideline count 99 -> 109 across metadata and platform
  template descriptions.
- Synced src/ -> cli/assets/ per the contributor guide.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@mrgoonie

Copy link
Copy Markdown
Contributor

Summary: I’m deferring this PR from the cron maintenance lane because it touches 43 files across both source and generated CLI asset copies. That is too broad for a bounded metadata-first cron review, even though the net diff is modest.

Decision: deferred / needs focused maintainer review

Evidence:

  • PR changes 43 files, including src/ source files, cli/assets/ mirrored files, README/skill metadata, and many platform template descriptions.
  • The change adds new UX guideline content plus generated/mirrored asset updates, so the review needs a source-of-truth check rather than only checking the GitHub diff.
  • Current cron policy defers PRs over 20 changed files to avoid shallow approvals of broad repo-wide changes.

Next step: please keep the PR narrow and make verification easy for a maintainer: identify the canonical source files, the exact command used to sync cli/assets, and the command/output proving guideline count + CSV/JSON validity. A maintainer should then review the data rows and generated mirrors together.

@mrgoonie mrgoonie added agent:github-maintain Processed by github-maintain automation maintain:deferred Deferred by maintain workflow pr:oversized Too large for cron-safe review budget labels Jun 20, 2026

@mrgoonie mrgoonie left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Approved from cron maintainer review.

Checked:

  • No duplicate PR/issue or prior merged commit found for Apple Essential Design Principles / WWDC 2017 additions.
  • The PR adds 10 UX rows to the canonical CSV and mirrored packaged CSVs; CSV parsing reports 109 rows and last row No=109.
  • skill.json parses as JSON.
  • python3 src/ui-ux-pro-max/scripts/search.py "apple design principles" --domain ux -n 10 returns the 10 new Apple Design Principles rows.
  • src/ui-ux-pro-max/data/ux-guidelines.csv matches cli/assets/data/ux-guidelines.csv; src/ui-ux-pro-max/templates/base/quick-reference.md matches the packaged base quick-reference.

Product fit is positive: this is small, source-backed foundational UX knowledge that improves concept-stage guidance without adding runtime or dependency surface.

@mrgoonie mrgoonie added maintain:blocked Blocked by external condition or maintainer action pr:blocked-conflict PR blocked by merge conflicts labels Jun 25, 2026
@mrgoonie

Copy link
Copy Markdown
Contributor

Maintainer merge attempt blocked: GitHub reports this PR is not currently mergeable because the merge commit cannot be cleanly created.

This PR was already approved for the current head (bc81f0ae00d397f8e405fdbcd7445df72b2ef8a1), but it now needs a rebase/merge of current main and conflict resolution before it can land.

Please rebase or merge main, resolve conflicts, and keep the source/mirror checks intact:

  • src/ui-ux-pro-max/data/ux-guidelines.csv matches cli/assets/data/ux-guidelines.csv
  • src/ui-ux-pro-max/templates/base/quick-reference.md matches cli/assets/templates/base/quick-reference.md
  • python3 src/ui-ux-pro-max/scripts/search.py "apple design principles" --domain ux -n 10 still returns the new Apple principle rows

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

agent:github-maintain Processed by github-maintain automation maintain:blocked Blocked by external condition or maintainer action maintain:deferred Deferred by maintain workflow pr:blocked-conflict PR blocked by merge conflicts pr:oversized Too large for cron-safe review budget

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants