fix(goldset): "tagged" = all 3 answers + rule-type help popover #102

chaim · 2026-06-07T13:27:12Z

chaim commented

2026-06-07 13:27:12 +00:00

מה ולמה

שני תיקוני-UX בדף מדגם-הזהב (משוב מחיים):

באג "מתויג": isTagged עכשיו דורש את שלושת השדות (is_holding וגם correct_type וגם quote_complete), לא רק is_holding. קודם, במצב "הסתר מתויגים", הכרטיס נעלם ברגע שסומן is_holding — ואי-אפשר היה להשלים את הסוג והציטוט. מד-ההתקדמות ותג "תויג" עכשיו משקפים תיוג מלא.
אייקון מידע (ℹ) ליד "הסוג הנכון" — בלחיצה נפתח popover עם טבלת ההסבר על 6 הסוגים (הגדרה + המבחן המכריע + דוגמה לכל אחד), כך שקריטריוני התיוג מול עיני המתייג.

אימות

tsc --noEmit exit 0.

🤖 Generated with Claude Code

## מה ולמה שני תיקוני-UX בדף מדגם-הזהב (משוב מחיים): 1. **באג "מתויג":** `isTagged` עכשיו דורש את **שלושת** השדות (`is_holding` **וגם** `correct_type` **וגם** `quote_complete`), לא רק `is_holding`. קודם, במצב "הסתר מתויגים", הכרטיס נעלם ברגע שסומן is_holding — ואי-אפשר היה להשלים את הסוג והציטוט. מד-ההתקדמות ותג "תויג" עכשיו משקפים תיוג **מלא**. 2. **אייקון מידע (ℹ) ליד "הסוג הנכון"** — בלחיצה נפתח popover עם טבלת ההסבר על 6 הסוגים (הגדרה + המבחן המכריע + דוגמה לכל אחד), כך שקריטריוני התיוג מול עיני המתייג. ## אימות - `tsc --noEmit` exit 0. 🤖 Generated with [Claude Code](https://claude.com/claude-code)

chaim added 1 commit 2026-06-07 13:27:12 +00:00

fix(goldset): "tagged" = all 3 answers set + add rule-type help popover 27911c5beb

Two UX fixes on the gold-set tagging page:

1. isTagged now requires is_holding AND correct_type AND quote_complete — not
   just is_holding. Previously, in "hide tagged" mode the card vanished the
   instant is_holding was clicked, so the type and quote-complete answers could
   never be set. The progress counter / "תויג" badge now reflect full tagging.

2. An info (ℹ) icon next to "הסוג הנכון" opens a popover explaining the six
   rule types (definition + the deciding test + an example each), so the tagger
   has the criteria in front of them while tagging.

Verified: tsc --noEmit exits 0.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

chaim merged commit b107654ee4 into main

2026-06-07 13:27:19 +00:00

chaim deleted branch worktree-goldset-tagged-fix

2026-06-07 13:27:19 +00:00

chaim referenced this issue from a commit

2026-06-07 13:27:21 +00:00

Merge pull request 'fix(goldset): "tagged" = all 3 answers + rule-type help popover' (#102) from worktree-goldset-tagged-fix into main

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: ezer-mishpati/legal-ai#102