[OPIK-3008] [FE] Refactor: NavigationTag infrastructure#3972
Merged
JetoPistola merged 2 commits intomainfrom Nov 9, 2025
Merged
[OPIK-3008] [FE] Refactor: NavigationTag infrastructure#3972JetoPistola merged 2 commits intomainfrom
JetoPistola merged 2 commits intomainfrom
Conversation
Contributor
There was a problem hiding this comment.
Pull Request Overview
This PR refactors the tooltip infrastructure for resource navigation tags by creating a reusable NavigationTag component that centralizes tooltip text generation. The main goal is to improve code maintainability by following DRY principles and establishing a consistent pattern for displaying navigable resource tags with tooltips.
Key changes:
- Extended
RESOURCE_MAPwith human-readablelabelfields for all resource types - Created new
NavigationTagcomponent that automatically generates standardized tooltip text - Refactored
AnnotationQueuePageto use the new component, eliminating manual tooltip content construction
Reviewed Changes
Copilot reviewed 4 out of 4 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| ResourceLink.tsx | Added label field to each resource type in RESOURCE_MAP and exported the map for external use |
| NavigationTag/index.ts | Created barrel export for the new NavigationTag component |
| NavigationTag/NavigationTag.tsx | Implemented new component that wraps ResourceLink with auto-generated tooltip text |
| AnnotationQueuePage.tsx | Replaced ResourceLink with NavigationTag, removing manual tooltip content generation |
Contributor
Collaborator
|
✅ Test environment is now available! Access Information
The deployment has completed successfully and the version has been verified. |
Contributor
Collaborator
|
✅ Test environment is now available! Access Information
The deployment has completed successfully and the version has been verified. |
YarivHashaiComet
approved these changes
Nov 9, 2025
vincentkoc
added a commit
that referenced
this pull request
Nov 10, 2025
into feat/optimizer-hybrid * 'feat/optimizer-hybrid' of https://github.com/comet-ml/opik: [OPIK-2986] [FE] Refactor comparison pages to use NavigationTag component (#4006) [OPIK-2992] [FE] Add tooltips to DateTag component (#4005) [OPIK-2993] [FE] Add tooltips to feedback scores and tags icons (#4007) [OPIK-3008] [FE] Refactor: NavigationTag infrastructure (#3972)
vincentkoc
added a commit
that referenced
this pull request
Nov 10, 2025
into vk/optimizer-oa_agent * 'vk/optimizer-oa_agent' of https://github.com/comet-ml/opik: [issue-2520] [SDK] Added Sycophancy Evaluation Metric (#2624) [OPIK-2986] [FE] Refactor comparison pages to use NavigationTag component (#4006) [OPIK-2992] [FE] Add tooltips to DateTag component (#4005) [OPIK-2993] [FE] Add tooltips to feedback scores and tags icons (#4007) [OPIK-3008] [FE] Refactor: NavigationTag infrastructure (#3972)
awkoy
pushed a commit
that referenced
this pull request
Nov 12, 2025
* [OPIK-3008] [FE] Refactor: Add NavigationTag infrastructure
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Details
Refactored tooltip infrastructure for resource navigation tags by creating a reusable
NavigationTagcomponent that generates consistent tooltip text across the application.Changes:
labelfield toRESOURCE_MAPinResourceLink.tsxfor all resource types (project, dataset, prompt, experiment, optimization run, trial, annotation queue)NavigationTagcomponent that automatically generates tooltip text: "Navigate to {resourceLabel}: {name}"AnnotationQueuePageto use the newNavigationTagcomponent, removing manual tooltip content generationImplementation:
src/components/shared/NavigationTag/NavigationTag.tsxid,name, andresourcepropsResourceLinkwithasTagprop for consistent stylingChange checklist
Issues
Testing
Documentation
No documentation update needed - internal frontend refactor.
Component usage pattern established for future implementations.