Repository navigation
NA-NATIVE-2: eight web-only properties are not applicable on native - #234
Open
thejackshelton wants to merge 2 commits into
Open
thejackshelton wants to merge 2 commits into
thejackshelton wants to merge 2 commits into
Conversation
-webkit-tap-highlight-color, -webkit-font-smoothing, text-size-adjust and -webkit-text-size-adjust, forced-color-adjust, -webkit-user-drag, and print-color-adjust and -webkit-print-color-adjust compile with an info diagnostic on iOS and Android and change nothing in the native output; web still refuses them. text-rendering and break-inside are pinned as off the list. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01JKaoLmr6Q9ij268L5imav2
Commands: pnpm regen --skip lanes-host --skip tw-sweep (macos-26) and pnpm regen --only lanes-host (xcode-27), alternated to a fixed point, then pnpm regen --only tw-sweep (macos-26). Base: 9e54843 Run: https://github.com/compiled-run/dragoncss/actions/runs/37898590741
This branch has not been deployed
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.
Requested by Jack · project thread
Before: a stylesheet using
-webkit-tap-highlight-color,-webkit-font-smoothing,text-size-adjust,forced-color-adjust,-webkit-user-dragorprint-color-adjustwas blocked on iOS and Android with DRAGON_UNSUPPORTED_PROPERTY.After: those properties (and the prefixed spellings
-webkit-text-size-adjustand-webkit-print-color-adjust) compile on iOS and Android with a DRAGON_NOT_APPLICABLE_NATIVE info, and the native output is byte-identical to the document without them. Web still refuses them, exactly as before.These are the NA-NATIVE-2 candidates from coverage-rank (#146) whose only effects are browser chrome, macOS text rendering, mobile-browser text inflation, forced-colors mode, mouse drag or printing. Left off on purpose and pinned as still refused:
text-rendering(optimizeSpeed turns off kerning and ligatures) andbreak-inside(it fragments columns, not only pages).speak,orphans,widows,page-break-*,sizeand@charset/@pageare not in this PR.How: eight entries in
profiles/not-applicable-native.ts, each with its reason; the existing per-entry tests innot-applicable-native.test.tscover every new entry (no error and one info per native target; native programs, Swift and Kotlin unchanged; web refusal unchanged).docs/api.mdnames the new list.The cheap
-webkit-alias work this lane was also asked about already landed (flex, box-sizing, logical and animation families). The aliases still missing (-webkit-border-radius,-webkit-transform,-webkit-background-*,-webkit-mask-*,-webkit-box-shadow,-webkit-user-select) wait on their property's lane, so this PR adds none.Owner review: the repo asks for per-entry owner review of NA-NATIVE-2 entries; the list was put to Jack in the project thread.
What passed (all on head ca838e4)
pnpm typecheck(local)packages/dragon/test/not-applicable-native.test.ts+packages/dragon/test/registry-claims.test.ts: 78 passed (local, platform-free)tailwind-sweep/test/units.test.ts37 passed,parity/test/chrome-ports.test.ts13 passed,parity/test/north-star-accounting.test.ts4 passed,parity/test/registry-claims.test.ts8 passedpnpm pr:review 234 --once: exit 0, UNREVIEWED (Macroscope at its monthly spending limit), 0 unanswered findingsTests changed
not-applicable-native.test.ts: the list pin grows by the eight names (the list is what changed); two off-list cases added. No check removed or loosened.🤖 Generated with Claude Code
https://claude.ai/code/session_01JKaoLmr6Q9ij268L5imav2
Generated by Claude Code