Skip to content

c30: HC Legendary Spirit window (CUIEnchantSkill) - #709

Merged
neeerp merged 1 commit into
masterfrom
v83/enchant-dialog
Oct 22, 2026
Merged

neeerp merged 1 commit into
masterfrom
v83/enchant-dialog

Conversation

@neeerp

@neeerp neeerp commented Oct 22, 2026

Copy link
Copy Markdown
Owner

Lane C slice c30 (LEDGER 3002, report v83ref/reports/c022-enchant-dialog.md).

HeavenClient gets Legendary Spirit's window (v83 CUIEnchantSkill on UIWindow.img/EnchantSkill), opened by the skill's key (1003, 10001003, 20001003):

  • An equip dropped in from the Equip tab (cash refused with "You may not use scrolls on this item." unless it is a pet's), then a scroll from the Use tab, sent at once as 0x56 {tick, scroll, +equip slot, ws 1/2, 1}.
  • The White Scroll box shows only while the player holds one.
  • The server's 0xA7 for the player's own legendary request goes to the window alone: -1 shows "You cannot use a Scroll with this item."; otherwise the Enchant sound and the window effect (26 frames), then the chat line, the scroll sound and glow. A cursed equip closes the window; otherwise it waits for the next scroll. Cancel/Close/Esc wait for the answer.
  • Sealed scrolls are refused; a sealed equip asks first; alerted/dead refusals as for normal scrolls.
  • Docs: agent_docs/inventory.md.

Verification (M): ctest 152/152 (scroll_rules tests for the new helpers), GUI check on the lane stack (window, drop, tooltip, White Scroll box, effect with Cancel disabled, chat line; DB shows the scroll and White Scroll used, slots kept), one Sonnet review (fixed: the chat line no longer tells of a White Scroll that wasn't sent; stale comment).

…p tab, a scroll from the Use tab sent as 0x56 with the White Scroll box; effect, sounds and chat line on the answer; -1 refused
@neeerp
neeerp merged commit b28ee6d into master Oct 22, 2026
1 check passed
@neeerp
neeerp deleted the v83/enchant-dialog branch October 22, 2026 06:31
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.

1 participant