Skip to content

feat: show ignore reason tooltip on IGNORE results#7

Merged
bartlomieju merged 1 commit into
mainfrom
show-ignore-reason
Mar 24, 2026
Merged

feat: show ignore reason tooltip on IGNORE results#7
bartlomieju merged 1 commit into
mainfrom
show-ignore-reason

Conversation

@bartlomieju
Copy link
Copy Markdown
Member

Summary

  • Add ignoreReason field to SingleResultOption type to match actual API data
  • Display a ? indicator next to IGNORE results that shows the reason on hover
  • Falls back to "(no reason specified)" when no reason is available

Test plan

  • Navigate to a day detail page with IGNORE results
  • Verify ? appears next to IGNORE text
  • Hover over IGNORE to see the reason tooltip
  • Verify tests without an ignore reason show "(no reason specified)"

🤖 Generated with Claude Code

Add ignoreReason field to SingleResultOption type and display it as a
hover tooltip with a ? indicator next to IGNORE results. Shows "(no
reason specified)" when no reason is available in the data.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@bartlomieju bartlomieju merged commit 0fe47b1 into main Mar 24, 2026
5 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.

1 participant