Skip to content

Fix entity-level reorg: match old event API for sidebar_tree.js#209

Merged
keyxmakerx merged 1 commit intomainfrom
claude/chronicle-launch-fixes-LcWMJ
Mar 31, 2026
Merged

Fix entity-level reorg: match old event API for sidebar_tree.js#209
keyxmakerx merged 1 commit intomainfrom
claude/chronicle-launch-fixes-LcWMJ

Conversation

@keyxmakerx
Copy link
Copy Markdown
Owner

sidebar_tree.js expects:

  • data-reorg-active attribute on #sidebar-entity-tree element
  • camelCase event detail: hiddenEntityIds, hiddenNodeIds (not snake_case)
  • HTMX afterSwap wait for lazy-loaded tree before dispatching event
  • 50ms setTimeout after swap for tree initialization

activateEntities() now:

  1. Sets data-reorg-active on #sidebar-entity-tree
  2. Sends camelCase hiddenEntityIds/hiddenNodeIds in event detail
  3. If tree not loaded yet, waits for htmx:afterSwap then retries
  4. Uses 50ms delay after swap for sidebar_tree.js init timing

deactivateEntities() now removes data-reorg-active from tree element.

This matches the exact behavior of the old sidebar_reorg.js that sidebar_tree.js was designed to work with.

https://claude.ai/code/session_01A5jpDgqUvW6iLXXSj49F27

sidebar_tree.js expects:
- data-reorg-active attribute on #sidebar-entity-tree element
- camelCase event detail: hiddenEntityIds, hiddenNodeIds (not snake_case)
- HTMX afterSwap wait for lazy-loaded tree before dispatching event
- 50ms setTimeout after swap for tree initialization

activateEntities() now:
1. Sets data-reorg-active on #sidebar-entity-tree
2. Sends camelCase hiddenEntityIds/hiddenNodeIds in event detail
3. If tree not loaded yet, waits for htmx:afterSwap then retries
4. Uses 50ms delay after swap for sidebar_tree.js init timing

deactivateEntities() now removes data-reorg-active from tree element.

This matches the exact behavior of the old sidebar_reorg.js that
sidebar_tree.js was designed to work with.

https://claude.ai/code/session_01A5jpDgqUvW6iLXXSj49F27
@keyxmakerx keyxmakerx merged commit e85c39c into main Mar 31, 2026
3 checks passed
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