Skip to content

feat: add logs content tooltip - #3267

Merged
seefs001 merged 1 commit into
QuantumNous:mainfrom
feitianbubu:pr/eb2ed7806ae5f2348681ce9fcff3f3572f535919
Mar 16, 2026
Merged

feat: add logs content tooltip#3267
seefs001 merged 1 commit into
QuantumNous:mainfrom
feitianbubu:pr/eb2ed7806ae5f2348681ce9fcff3f3572f535919

Conversation

@feitianbubu

@feitianbubu feitianbubu commented Mar 16, 2026

Copy link
Copy Markdown
Member

修复使用日志当详情为普通文本的时候, tooltip因重构丢失的问题
image

Summary by CodeRabbit

  • New Features
    • Added hover tooltips to display truncated detail text in full, enabling users to preview complete information without modifying the interface.

@coderabbitai

coderabbitai Bot commented Mar 16, 2026

Copy link
Copy Markdown
Contributor

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: 24b86551-212f-4886-9b4c-2b731eccccf7

📥 Commits

Reviewing files that changed from the base of the PR and between 5a7b72e and 9984390.

📒 Files selected for processing (1)
  • web/src/components/table/usage-logs/UsageLogsColumnDefs.jsx

Walkthrough

Adds tooltip configuration to the ellipsis property in the DETAILS column of a Usage Logs table component, enabling a popover that displays truncated detail text with a fixed width of 240 pixels.

Changes

Cohort / File(s) Summary
Tooltip Enhancement
web/src/components/table/usage-logs/UsageLogsColumnDefs.jsx
Adds showTooltip configuration to the ellipsis prop of a Typography.Paragraph element in the DETAILS column definition, enabling a styled tooltip for truncated text when no detail summary is available.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~7 minutes

Poem

🐰 A tooltip now appears when text gets too small,
A whisper of details in a popover call,
Width set just right, no more mystery—
Our table's now kinder to those who must see! ✨

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title 'feat: add logs content tooltip' clearly and directly describes the main change—adding a tooltip feature for logs content, which matches the core functionality added (showTooltip configuration for ellipsis in the details column).
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

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

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
📝 Coding Plan
  • Generate coding plan for human review comments

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.

Tip

You can validate your CodeRabbit configuration file in your editor.

If your editor has YAML language server, you can enable auto-completion and validation by adding # yaml-language-server: $schema=https://coderabbit.ai/integrations/schema.v2.json at the top of your CodeRabbit configuration file.

@seefs001
seefs001 merged commit 0eee341 into QuantumNous:main Mar 16, 2026
1 check passed
ennnnny pushed a commit to ennnnny/new-api that referenced this pull request Mar 17, 2026
…2348681ce9fcff3f3572f535919

feat: add logs content tooltip
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