hechang27
/
noob-components
forked from mengyxu/noob-components
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity
基于vue3.0和element-plus的组件库
You can not select more than 25 topics Topics must start with a letter or number, can include dashes ('-') and can be up to 35 characters long.
126 Commits
2 Branches
0 Tags
6.1 MiB
 
 
 
 
Tree: 3a3652ed1f
Branches Tags
${ item.name }
Create tag ${ searchTerm }
Create branch ${ searchTerm }
from '3a3652ed1f'
${ noResults }
noob-components/.trellis/workspace/hechang27-sprt/journal-1.md

14 KiB
Raw Blame History

Journal - hechang27-sprt (Part 1)

AI development session journal Started: 2026-03-24

Session 1: Bootstrap coding specs with CC + Codex pipeline

Date: 2026-03-24 Task: Bootstrap coding specs with CC + Codex pipeline

Summary

Bootstrap session completed all 8 tasks: (1) Bootstrap Guidelines - set up Trellis framework with Claude Code hooks and multi-agent pipeline, (2) All 7 spec-filling tasks completed - plugs/composables (7 composables), plugs/api (9 modules), plugs/element (4 docs), plugs/http (4 docs), packages/base (18 components), packages/manage (views/head/common), plugs/config+store+i18n (7 docs). All specs contain real code examples with file paths, anti-patterns documented, no placeholder text.

Main Changes

Git Commits

Hash Message
785e6b2 (see git log)

Testing

  • [OK] (Add test results)

Status

[OK] Completed

Next Steps

  • None - task complete

Session 2: Create list-table-v2.vue with el-table-v2

Date: 2026-03-25 Task: Create list-table-v2.vue with el-table-v2

Summary

Implemented list-table-v2.vue using Element Plus el-table-v2 with virtual scrolling. Key findings: (1) el-table-v2 requires ElAutoResizer wrapper for auto-sizing, (2) slot-to-cellRenderer conversion via renderSlot, (3) fixed import path issue in useListTable.ts and plugs/store/index.ts. Updated component-guidelines.md with el-table-v2 usage notes. Added demo page at examples/view/base/table-v2.vue with 100 rows of test data.

Main Changes

Git Commits

Hash Message
2a2bba8 (see git log)
1024bb1 (see git log)

Testing

  • [OK] (Add test results)

Status

[OK] Completed

Next Steps

  • None - task complete

Session 3: Add viewport resize handling for list-table-v2

Date: 2026-03-26 Task: Add viewport resize handling for list-table-v2

Summary

Implemented debounced window resize handler (250ms) to re-measure probe row height when viewport changes. Only active in dynamic height mode (when rowHeight is undefined). Properly cleans up on unmount. Fixed TypeScript typing for lodash debounce cancel method.

Main Changes

Git Commits

Hash Message
96fef48 (see git log)

Testing

  • [OK] (Add test results)

Status

[OK] Completed

Next Steps

  • None - task complete

Session 4: list-table-v2 mini table for row height with ResizeObserver

Date: 2026-03-26 Task: list-table-v2 mini table for row height with ResizeObserver

Summary

Implemented hidden mini table approach for dynamic row height measurement in list-table-v2. Key improvements: (1) Replaced window resize with ResizeObserver on container to prevent flickering, (2) Mini table uses 3-5 sample rows with same JSX rendering as real table (slots, TzDateTime, dicts), (3) Header rendering added with dynamic header height measurement, (4) Extracted shared renderCellContent function to avoid code duplication, (5) Fixed header text wrapping to allow multi-line headers

Main Changes

Git Commits

Hash Message
bd47429 (see git log)
c9f8832 (see git log)
e72f42a (see git log)
1fe3718 (see git log)
ed00f0b (see git log)
854a8f6 (see git log)
242f376 (see git log)

Testing

  • [OK] (Add test results)

Status

[OK] Completed

Next Steps

  • None - task complete

Session 5: Fix table-v2 resize flicker with queueMicrotask pattern

Date: 2026-03-26 Task: Fix table-v2 resize flicker with queueMicrotask pattern

Summary

Fixed 1-2 frame flash during window resize by using queueMicrotask to minimize gap between clearing and setting estimatedRowHeight. Also documented build commands and rename propagation gotcha in spec guides.

Main Changes

Git Commits

Hash Message
179aafe (see git log)

Testing

  • [OK] (Add test results)

Status

[OK] Completed

Next Steps

  • None - task complete

Session 6: Expose cellRenderer and headerCellRenderer for list-table-v2

Date: 2026-03-27 Task: Expose cellRenderer and headerCellRenderer for list-table-v2

Summary

Added cellRenderer and headerCellRenderer column attributes to list-table-v2. Fixed mini-table VNode rendering using functional components. Fixed CSS for accurate row height measurement. Added ts-pattern dependency.

Main Changes

Git Commits

Hash Message
3c97cd7 (see git log)

Testing

  • [OK] (Add test results)

Status

[OK] Completed

Next Steps

  • None - task complete

Session 7: Restructure and cleanup list-table-v2 mini-table rendering

Date: 2026-03-28 Task: Restructure and cleanup list-table-v2 mini-table rendering

Summary

Refactored mini-table rendering to use useTemplateRef, added debug display for height estimation, fixed md5 import bug, and fixed loading state bug in router getUser callback

Main Changes

Git Commits

Hash Message
a2eeb86 (see git log)

Testing

  • [OK] (Add test results)

Status

[OK] Completed

Next Steps

  • None - task complete

Session 8: Table-v2 comprehensive demo page

Date: 2026-03-30 Task: Table-v2 comprehensive demo page

Summary

(Add summary)

Main Changes

Session Summary

Reworked examples/view/base/table-v2.vue into a comprehensive demo page for list-table-v2, patterned after Element Plus documentation style.

Features Implemented

# Section Features Tested
1 Basic Usage Data binding, columns, pagination, @query/@row-click/@cell-click events
2 Styling border prop, align (left/center/right)
3 Fixed Columns fixed: "left"/"right"
4 Data Formatting timestamp: "unix", filesize: true, dict
5 Custom Cell Renderer cellRenderer JSX function
6 Custom Header Renderer headerCellRenderer JSX function
7 Dynamic Height Auto-probe (no rowHeight), ResizeObserver, debug
8 Fixed Height Mode rowHeight: 60 (disables probe)
9 Column Width width, minWidth
10 Dimensions height, maxHeight, headerHeight
11 Slot Cells slot: true + #columnKey template
12 i18n Support i18n key with vue3-i18n
13 Combined Test All features together stress test
14 Renderer Edge Cases cellRenderer + fixed + pagination

Bug Fixes

  • Fixed headerCellRenderer shorthand property error (was using wrong variable name)
  • Added missing toggleMaxHeight function
  • Fixed data format: page="false" tables need plain arrays, not {data, total} objects
  • Fixed .app-main overflow to enable page scrolling

Updated Files

  • examples/view/base/table-v2.vue - Demo page (14 sections, ~1000 lines)
  • packages/manage/router/index.vue - Changed overflow: hidden → overflow-y: auto

Verification

  • Build passes: bun run build ✓
  • TypeScript passes: tsc --noEmit ✓
  • Browser tested: All 14 tables render data correctly, no JS errors

Git Commits

Hash Message
62ab397 (see git log)

Testing

  • [OK] (Add test results)

Status

[OK] Completed

Next Steps

  • None - task complete

Session 9: Optimize list-table-v2 resize performance

Date: 2026-03-31 Task: Optimize list-table-v2 resize performance

Summary

Implemented 3-layer performance optimization for list-table-v2: (1) Height-change detection - only re-probe when mini-table height actually changes, (2) Viewport-based observer detachment - ResizeObserver only attached when table is visible, (3) Display:none for off-screen tables - removes hidden mini-tables from GPU layout. Also added oxlint as linter.

Main Changes

Git Commits

Hash Message
5e88867 (see git log)

Testing

  • [OK] (Add test results)

Status

[OK] Completed

Next Steps

  • None - task complete

Session 10: Separate CSS classes for mini-table and real table cells

Date: 2026-04-01 Task: Separate CSS classes for mini-table and real table cells

Summary

(Add summary)

Main Changes

Change Description
Refactor Separated CSS classes for mini-table (height measurement) vs real table (el-table-v2) cell rendering

Changes Made:

  • packages/base/data/list-table-v2.vue: Added isMiniTable parameter to renderCellContent and renderHeaderCellContent to distinguish rendering context
  • Mini-table cells use mini-cell-text / mini-header-cell-text (unchanged - allows natural growth for height measurement)
  • Real table cells use new table-cell-text / table-header-cell-text classes (with overflow: hidden; text-overflow: ellipsis)

Purpose: This is a prerequisite for fixing the text cropping bug in fixed columns. The real table cells need different CSS constraints than the mini-table which is used purely for measuring row heights.

Git Commits

Hash Message
423d7ef (see git log)

Testing

  • [OK] (Add test results)

Status

[OK] Completed

Next Steps

  • None - task complete

Session 11: Research Pretext.js integration approach

Date: 2026-04-01 Task: Research Pretext.js integration approach

Summary

(Add summary)

Main Changes

Change Description
Research Studied Pretext.js documentation for text height calculation without DOM
Analysis Identified width calculation as key challenge for flex columns

Pretext.js Research Findings:

  1. API: prepare(text, font) + layout(handle, width, lineHeight) = height without DOM
  2. Performance: ~500x faster than DOM measurement for batch operations

Width Challenge Identified:

  • el-table-v2 uses CSS flexbox (flexGrow, flexShrink, flexBasis)
  • Actual pixel widths depend on browser's flexbox algorithm
  • Cannot calculate without replicating flex algorithm

Solution Developed:

  • Created useColumnWidthCalculator.ts to mathematically replicate CSS flexbox distribution
  • Formula: width = flexBasis + (freeSpace * flexGrow / totalFlexGrow) for growing
  • Handles shrinking case and minWidth/maxWidth clamping

Attempted Integration (Failed):

  • Added containerWidth ref, columnWidthCalc, and updateContainerWidth function
  • Got "Cannot access 'tableColumns' before initialization" error
  • Error persisted even with getter function approach
  • Reverted changes to restore working state

For Next Session:

  1. Debug the initialization order issue with useColumnWidthCalculator
  2. Alternatively: integrate Pretext directly into renderCellContent without separate composable
  3. Test width calculation accuracy against actual el-table-v2 rendering

Git Commits

Hash Message
423d7ef (see git log)

Testing

  • [OK] (Add test results)

Status

[OK] Completed

Next Steps

  • None - task complete

Session 12: Pretext integration exploration - virtual table issues

Date: 2026-04-02 Task: Pretext integration exploration - virtual table issues

Summary

(Add summary)

Main Changes

Session Summary

Attempted to integrate Pretext.js into list-table-v2.vue for synchronous row height calculation without DOM reflow.

What Was Done

  1. Created useColumnWidthCalculator.ts - Composable that mathematically replicates CSS flexbox column width distribution for el-table-v2

  2. Created usePretextHeight.ts - Composable for calculating text heights using Pretext.js with caching

  3. Attempted VirtualTable approach - Created a functional JSX component inside list-table-v2 that:

    • Receives width directly from el-auto-resizer slot props
    • Calculates estimatedRowHeight using Pretext synchronously during render
    • Uses toRaw() to break reactive chains and avoid recursion

  4. Issues encountered:

    • Recursion error: onItemRendered callback in el-table-v2 was triggering reactive loops
    • Fixed by removing nested computed and using toRaw()
    • Virtual row count bug: el-table-v2 only rendered 3 rows instead of ~10 with estimatedRowHeight set
    • The VirtualTable approach caused el-table-v2's DynamicSizeGrid to miscalculate the virtual scroll window

  5. Root cause: el-table-v2's DynamicSizeGrid uses estimatedRowHeight to calculate how many rows to render in the viewport. With actual rows at 28px but estimated heights causing miscalculations, the virtual scroller wasn't showing all rows.

  6. Reverted changes: Reverted list-table-v2.vue to original code since the VirtualTable approach wasn't working correctly.

Key Files Created

  • packages/base/data/useColumnWidthCalculator.ts - Flex column width calculator
  • packages/base/data/usePretextHeight.ts - Pretext height calculator composable

Lessons Learned

  1. el-table-v2's virtual scrolling is sensitive to estimatedRowHeight accuracy
  2. CSS variables don't work in Canvas (Pretext) context - must use plain font specs
  3. Reactive loops can occur when el-table-v2 callbacks trigger Vue reactivity
  4. toRaw() helps break reactivity but isn't a complete solution
  5. The original mini-table DOM measurement approach works correctly (10 rows showing)

Next Steps

  • Investigate why el-table-v2's DynamicSizeGrid miscalculates with Pretext-estimated heights
  • Consider using Pretext only for initial estimate, then relying on mini-table for refinement
  • Or find a different integration point that doesn't conflict with el-table-v2's internal calculations

Git Commits

Hash Message
0764ee2 (see git log)

Testing

  • [OK] (Add test results)

Status

[OK] Completed

Next Steps

  • None - task complete

Session 13: Attempt Pretext row height estimation for el-table-v2

Date: 2026-04-02 Task: Attempt Pretext row height estimation for el-table-v2

Summary

Tried to integrate Pretext.js for DOM-less row height estimation. Key finding: el-table-v2 caches row heights internally and only re-measures when rows scroll into view. The estimatedRowHeight prop doesn't trigger re-render of existing rows. The table's virtualization cache persists even when destroying/recreating the component. This approach needs more investigation - possibly requiring deeper integration with el-table-v2's internal measurement system or using the table's built-in dynamic height mechanism differently.

Main Changes

Git Commits

Hash Message
stashed (see git log)

Testing

  • [OK] (Add test results)

Status

[OK] Completed

Next Steps

  • None - task complete