Skip to content

Remove duplicate skill text from draft confirmation

Idan Cohen requested to merge fix/concise-draft-confirmation into genocide

The draft confirmation repeated the full description, level-one effects and HP/MP cost block already available in the skill tooltip. It now shows the skill name and useful weapon, prerequisite-progress or special-content notes, followed by Learn/Back.

The Learn row carries the selected skill icon and hover metadata, keeping the full tooltip available during confirmation. Its existing 32-bit action ID still performs the server-authoritative level-one grant with maximum mastery cap.

Validation: the existing isolated draft/classless suite passes, including 955 running-server assertions covering first/reopened selections, Back, cancellation, claim failures, commit reconciliation and save/reload. Two offline X11 rendered cases consume the actual NPC script output using production client source matching v1.0.7: both actions fit without scrolling, hover retains the full description, and leaving clears it.

Rendered fixtures: compact confirmation, hover tooltip.

Related to #10 (closed). Server-only change; no client release required. Ready for owner review and merge; not deployed.

Merge request reports