Skip to content

device clean#29261

Merged
chrisnojima merged 12 commits into
nojima/HOTPOT-next-670-clean-2from
nojima/HOTPOT-device-clean
May 29, 2026
Merged

device clean#29261
chrisnojima merged 12 commits into
nojima/HOTPOT-next-670-clean-2from
nojima/HOTPOT-device-clean

Conversation

@chrisnojima
Copy link
Copy Markdown
Contributor

No description provided.

This comment was marked as outdated.

Documents which style properties can be lifted into Box2 props (purely
structural, no visual change) and how to use gap/gapStart/gapEnd to
replace per-child margins (slight visual change — must be validated with
user before applying). Also relaxes the hard-line no-visual-changes rule
to allow validated visual changes with explicit user sign-off.
…ces/

- Extract box2ClassNames() from Box2 desktop branch into shared helper
- Export box2SharedProps from box.tsx
- ClickableBox3Props = Box2Props & {onClick?, onLongPress?, hitSlop?}
- CB3 desktop: box2ClassNames + clickable-box2 CSS class
- CB3 mobile: box2SharedProps + Pressable
- Migrate devices/ CB1/CB2 → CB3, eliminate inner Box2 wrappers
- Simplify mobileAddHeader style (flex/position props move to CB3 props)
- Add plans/clickablebox3.md migration plan with per-directory checklist
- Add/update migrate-clickable-box skill for CB3 migration
Replace fixed-height virtualized List with ScrollView so long TLF names that wrap to multiple lines don't overlap adjacent rows.
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 17 out of 17 changed files in this pull request and generated no new comments.

@chrisnojima chrisnojima merged commit 4a07f73 into nojima/HOTPOT-next-670-clean-2 May 29, 2026
1 of 2 checks passed
@chrisnojima chrisnojima deleted the nojima/HOTPOT-device-clean branch May 29, 2026 15:59
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