Skip to content

EPMRPP-117356 || Update accentToolTip for pagination#307

Merged
siarheirazuvalau merged 1 commit into
developfrom
bugfix/EPMRPP-117356
Jul 6, 2026
Merged

EPMRPP-117356 || Update accentToolTip for pagination#307
siarheirazuvalau merged 1 commit into
developfrom
bugfix/EPMRPP-117356

Conversation

@siarheirazuvalau

@siarheirazuvalau siarheirazuvalau commented Jul 6, 2026

Copy link
Copy Markdown
Contributor

Description

Added maximum width for accent tooltip
image

Summary by CodeRabbit

  • Bug Fixes
    • Improved the pagination item counter tooltip so long content displays more comfortably and is less likely to overflow.
    • Updated tooltip styling for the total count display to better fit the available space.

@coderabbitai

coderabbitai Bot commented Jul 6, 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: 0f14fad6-e61c-49da-aa67-ca95b0a90963

📥 Commits

Reviewing files that changed from the base of the PR and between c8b1ca7 and 118e0d7.

📒 Files selected for processing (2)
  • src/components/pagination/itemsCounter/itemCounter.module.scss
  • src/components/pagination/itemsCounter/itemCounter.tsx

Walkthrough

Adds a new SCSS rule .accent-total-tooltip limiting tooltip max-width to 300px, and updates the Tooltip usage in itemCounter.tsx to pass dynamicWidth and tooltipClassName props for the accent total tooltip rendering branch.

Changes

Accent Total Tooltip Styling

Layer / File(s) Summary
Tooltip sizing and props
src/components/pagination/itemsCounter/itemCounter.module.scss, src/components/pagination/itemsCounter/itemCounter.tsx
Adds .accent-total-tooltip class capping max-width at 300px and wires it via tooltipClassName plus a new dynamicWidth prop on the Tooltip component.

Estimated code review effort: 1 (Trivial) | ~5 minutes

Possibly related PRs

  • reportportal/ui-kit#305: Both PRs modify the accentTotalTooltip rendering path in itemCounter.tsx and its tooltip styling in itemCounter.module.scss.

Suggested reviewers: VaheSamsonyan, allaprischepa

Poem

A tooltip once stretched far and wide,
Now bounded neatly, three-hundred px tide 🐇
With dynamic width and a class of its own,
This little rabbit's hop is nicely shown!
Hop, hop, hooray for tidy CSS! 🥕

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title is clearly related to the main change: updating the accent tooltip for pagination.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
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.
✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch bugfix/EPMRPP-117356

Warning

Tools execution failed with the following error:

Failed to run tools: 13 INTERNAL: Received RST_STREAM with code 2 (Internal server error)


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.

@siarheirazuvalau siarheirazuvalau merged commit d7e6660 into develop Jul 6, 2026
2 checks passed
@siarheirazuvalau siarheirazuvalau deleted the bugfix/EPMRPP-117356 branch July 6, 2026 19:18
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