Open
Conversation
TUI now handles text addon styling natively via .is-text modifier — surface background, border separators, and group padding adjustment are all handled upstream. Scope legacy input styles to .tf-text-legacy. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Button: add data-test-id and aria-label props, extract ForwardedTuiProps type, fix LegacyButton ref binding, add isDevBuild check. Imports: convert barrel imports to direct file paths across dynamic components to avoid circular dependencies. Config: disable Storybook react-docgen, bump @tangible/ui to ^0.0.3, add wp.scss control-height and Button typography tokens. Examples: add dynamic value registration and ComboBox layout demos. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
ProseMirror suite: schema, tokeniser, serialiser, node views, and plugins (single-line, replace-on-type, dedup-ids, placeholder) for editing text with embedded [[dynamic::value]] tokens. DynamicEditor: React component lazy-loaded by text/textarea fields, manages ProseMirror lifecycle with imperative insert/update API. DynamicFieldSettings: modal for inserting/editing dynamic values with built-in (ComboBox picker) and custom (free-text) modes. Utilities: input masking hook (digit/letter/alphanumeric patterns with IME and Android support), mask validation, and ID generation. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Text field now routes between dynamic (ProseMirror), masked (input mask hook), and plain (TUI TextInput) paths. Textarea gains the same dynamic routing for multi-line mode. Uses TUI Field compound component for consistent ARIA wiring. Adds Text.stories.tsx covering all three code paths. Includes build output. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
…d import isDev Replace hasWarnedAboutSpanButton/hasWarnedAboutMissingLabel module-level singletons with per-instance useRef(false) guards. Replace hand-rolled isDevBuild const with shared isDev() utility. Add onChange dev warning via useEffect+useRef guard. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
…t, expand span fragility comment Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Add typed interface covering all used props (dynamic, inputMask, prefix, suffix, value, onChange, name, placeholder, readOnly, isDisabled, isRequired, isInvalid, error, label, labelVisuallyHidden, description, descriptionVisuallyHidden, className, inputClassName, size, type). Import SizeStandard and TextInputType from @tangible/ui for accurate size/type prop types. Fix useRef() no-arg call and cast useTextField args (both will be properly refactored in tasks 5-6). Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Move useTextField (react-aria) into an internal DynamicTextField component so it is only called on the dynamic/ProseMirror path. Wrap TextField with forwardRef<HTMLInputElement> and pass the forwarded ref to TuiTextInput in the TUI path via a callback ref that also handles inputMaskRef. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
…elperText comment
TUI TextInput forwards its ref directly to the <input> element, so
node.querySelector('input') is unnecessary. Also adds an explanatory
comment on the tui-visually-hidden className workaround for Field.HelperText.
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Add Jest tests for: Button with mapped layout (primary) renders TUI Button; Button with unmapped layout falls back to LegacyButton; Text without dynamic prop renders TUI TextInput (.tf-text-tui + input); Text onChange does not fire on initial mount. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
- Add @babel/preset-typescript with allExtensions/isTSX for .ts files with JSX - Add custom babel-plugin-import-meta-env to shim import.meta.env for CJS - Configure transformIgnorePatterns to transform @tangible/ui through Babel - Add moduleNameMapper for Jest 29 package exports compatibility - Explicit transform config so Babel applies to node_modules too - Bump @tangible/ui from ^0.0.3 to ^0.0.4 Goes from 0/33 suites passing to 19/33 (333 tests passing). Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Harness config, build prompt, specs (wrapper-contract, fix-button, fix-text), plan, and progress log. All 8 initial tasks complete. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
…pound Replace react-aria useCheckbox/useToggleState/useField with TUI Checkbox. Add FieldsCheckboxProps interface, forwardRef<HTMLInputElement>, Field compound wrapper. Map isDisabled→disabled, onChange→onCheckedChange. Preserve hidden input for PHP form submission. Sync external value changes (repeater bulk-select) via useEffect without calling onChange. Add render-time label warning to maintain a11y parity with react-aria. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Remove local checked state and value-sync useEffect. Derive boolValue
directly from the value prop — when provided, pass as controlled
checked={boolValue}; when absent, use defaultChecked={false}. Simplifies
onCheckedChange to call onChange directly without setChecked. No
mount-fire bug in TUI path (onCheckedChange is interaction-only).
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Replace react-aria useSwitch/useToggleState/useFocusRing with TUI Switch. Add FieldsSwitchProps interface, forwardRef<HTMLButtonElement>, hidden input. Use useState + useEffect for controlled value sync from value prop. Update FieldGroup test to click [role="switch"] instead of .tf-switch-label. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Remove local checked state and useEffect value sync. Derive boolValue
directly from value prop; use checked={boolValue} when controlled,
defaultChecked={false} otherwise. TUI onCheckedChange is
interaction-only so no mountedRef guard needed.
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
… compound Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Replace react-aria useRadio + RadioContext with TUI Radio. Add FieldsRadioProps interface, forwardRef<HTMLButtonElement>. Map children → TUI label prop. Remove RadioContext stub from RadioGroup. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
…pound Extract DynamicTextarea sub-component (useTextField isolated to dynamic path). Replace native textarea path with TUI Textarea + Field compound. Add FieldsTextareaProps interface, forwardRef<HTMLTextAreaElement>. TUI native onChange maps event.target.value to Fields onChange(value). Preserves data-identifier via TextareaHTMLAttributes spread. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
…hange on mount Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
…pping Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
…tarea, Notice - Augment Checkbox.test.tsx with TUI path tests (onChange-on-mount, controlled value, disabled) - Add Switch.test.tsx: renders tf-switch + [role="switch"], onChange guards, aria-checked, disabled - Add Radio.test.tsx: renders tf-radio-group, options, onChange guards, disabled - Add Textarea.test.tsx: renders tf-text-area + textarea, controlled value, readOnly - Add Notice.test.tsx: renders tf-notice, message content, type→theme mapping, dismissible toggle Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
…a, Notice Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Add @floating-ui/react and @tanstack/react-table required by TUI 0.0.8. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
TUI Button uses native onClick, but react-aria hooks (useComboBox, useDatePicker) spread buttonProps expecting PressEvent-based handlers. Synthesize a PressEvent from the click and forward it to onPressStart and onPress. Also strip low-level press/focus handlers to prevent double-firing. Without this, ComboBox dropdowns, DatePicker calendars, and calendar navigation buttons silently stop responding to clicks. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Without this, pointerdown closes the popover then click reopens it. Needed since TUI migration: TUI Button uses native onClick instead of react-aria's useButton, so usePopover and the trigger no longer share the same event pipeline. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Source fixes: - ComboBox: respect menuTrigger prop instead of hardcoding 'input' - DynamicFieldSettings: import ComboBox from index wrapper (adds menuTrigger="focus", choices→defaultItems, RenderChoices) - DynamicFieldSettings: add isDismissable to useModalOverlay - Store: actually set value in setValue, trigger valueChange event Test fixes: - Update legacy selectors to TUI class names across all test files - Handle text type lazy-loading DynamicEditor (async waitFor, warmup) - Add ComboBox focus step for text type dynamic modal - Fix Radio disabled assertion (aria-disabled instead of native) - Broaden RepeaterTable label filter for TUI visually-hidden classes Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
…enderWrapper to fire setValue
…sed as Field and Element)
…works outside of wordpress
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.
Adds upgraded TUI Components for: