Tooltip to show feed scores on frontend#776
Tooltip to show feed scores on frontend#776regulartim merged 8 commits intointelowlproject:developfrom
Conversation
There was a problem hiding this comment.
Pull request overview
This PR adds a tooltip/popover feature to display feed scores and detailed information on the frontend, consolidating the Scanner and Payload Request columns into a single Details column with an info icon that reveals comprehensive data on hover.
Changes:
- Added popover styling in SCSS to match the application's dark theme
- Consolidated Scanner and Payload Request columns into a new Details column with an interactive info button
- Implemented helper functions to format score data (integers and percentages) with proper null/undefined handling
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 3 comments.
| File | Description |
|---|---|
| frontend/src/styles/App.scss | Added .feeds-details-popover styling for dark-themed popover display |
| frontend/src/components/feeds/tableColumns.jsx | Replaced separate Scanner and Payload Request columns with a Details column featuring a popover that displays scores, activity metrics, enrichment data, and flags |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
regulartim
left a comment
There was a problem hiding this comment.
Hey @armoredvortex ! Thanks for contributing. Looks very nice so far. Do you think it is better to put the button that triggers the pop-up next to the table instead of giving it a whole column?
|
@regulartim |
regulartim
left a comment
There was a problem hiding this comment.
Hey @armoredvortex ! I am not a frontend developer so I trust you that the button next to the table would look weird. Other than one minor detail we need to discuss, this is ready to get merged. 👍
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 2 out of 2 changed files in this pull request and generated 6 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
@regulartim I believe I've covered all of the requested changes, do you also want me to write the jest tests as suggested by copilot review? |
Thank you! :) Yes, please add tests as suggested. Otherwise, this is ready to be merged. Good work! 👍 |
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
I've added a simple test that clicks on the button, and checks if the popover is displayed. |
Description
Display scores in feed on frontend.
Related issues
#461
Type of change
Checklist
develop.Ruff) gave 0 errors. If you have correctly installed pre-commit, it does these checks and adjustments on your behalf.Screenshot