Skip to content

Kpmp 6691 edit diffex tooltips#647

Merged
HaneenT merged 2 commits into
developfrom
KPMP-6691_edit-diffex-tooltips
Jun 5, 2026
Merged

Kpmp 6691 edit diffex tooltips#647
HaneenT merged 2 commits into
developfrom
KPMP-6691_edit-diffex-tooltips

Conversation

@Dert1129

@Dert1129 Dert1129 commented Jun 5, 2026

Copy link
Copy Markdown
Contributor

Summary by CodeRabbit

  • Style
    • Enhanced tooltips for differential expression metrics with clearer descriptions explaining how calculations are performed
    • Improved tooltip visual styling for better readability and presentation

@coderabbitai

coderabbitai Bot commented Jun 5, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: cdf5a32e-66b0-4bb1-a89e-5b13be04a3fb

📥 Commits

Reviewing files that changed from the base of the PR and between bcc463e and 4e2d509.

📒 Files selected for processing (2)
  • src/components/ExpressionTables/DiffexByCluster.js
  • src/index.scss

Walkthrough

This PR improves tooltip readability in the DiffexByCluster component by reformatting column header tooltips as multi-line template literals and adding CSS styling to render them correctly. The FOLD CHANGE and P VALUE column descriptions are now split across multiple lines, and the stylesheet includes new .ag-tooltip styling with white-space: pre-line to preserve line breaks.

Changes

Tooltip display improvements

Layer / File(s) Summary
Tooltip styling and content updates
src/index.scss, src/components/ExpressionTables/DiffexByCluster.js
.ag-tooltip CSS styling with white-space: pre-line is added to support multi-line tooltip display. The FOLD CHANGE and P VALUE column header tooltips are reformatted from single-line strings to multi-line template literals, preserving their meaning while improving readability across multiple lines.

Possibly related PRs

  • KPMP/pegasus-web#631: Both PRs modify DiffexByCluster column header tooltips; retrieved PR adds a new COMPARISON column tooltip while this PR improves FOLD CHANGE and P VALUE tooltip formatting.
🚥 Pre-merge checks | ✅ 2
✅ Passed checks (2 passed)
Check name Status Explanation
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch KPMP-6691_edit-diffex-tooltips

Warning

Review ran into problems

🔥 Problems

Git: Failed to clone repository. Please run the @coderabbitai full review command to re-trigger a full review. If the issue persists, set path_filters to include or exclude specific files.


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@HaneenT HaneenT merged commit d3c70f6 into develop Jun 5, 2026
1 check passed
@HaneenT HaneenT deleted the KPMP-6691_edit-diffex-tooltips branch June 5, 2026 14:38
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.

2 participants