EPMRPP-118299 || "Play"and "Full screen" icons in the video player on the "Remote device" tab are not visible for 300px width reopened

[VaheSamsonyan]

Visuals

colfix.mp4

Summary by CodeRabbit

• Bug Fixes
  • Improved remote device panel resizing with separate minimum widths for the left and right panels.
  • Reduced the left panel’s minimum width to 300px while maintaining a 400px minimum for the right panel.
  • Prevented panels from being resized below their designated limits.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 8ea6d38d-ad21-422b-bcfb-9f1fa5463ff8

📥 Commits

Reviewing files that changed from the base of the PR and between f4d7437 and 821c3ef.

📒 Files selected for processing (2)

• ui/src/components/remoteDeviceTab/remoteDeviceTab.scss
• ui/src/components/remoteDeviceTab/remoteDeviceTab.tsx

---

Walkthrough

The remote device tab now allows the left panel to shrink to 300px. The right panel keeps a 400px minimum. Responsive resizing and pointer dragging enforce these separate limits.

Changes

Panel resizing

Layer / File(s)	Summary
Separate panel minimum widths ui/src/components/remoteDeviceTab/remoteDeviceTab.tsx, ui/src/components/remoteDeviceTab/remoteDeviceTab.scss	The left panel uses a 300px minimum. The right panel uses a 400px minimum during layout resizing and pointer dragging.

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

Possibly related PRs

• reportportal/plugin-mobitru#37: Introduced the panel resizing implementation updated by this change.
• reportportal/plugin-mobitru#40: Changed the shared panel minimum before this change restored separate limits.

Suggested reviewers: allaprischepa, siarheirazuvalau

Poem

A rabbit resized the panel with care,
Three hundred pixels now fit there.
Four hundred stays on the other side,
While smooth drag limits safely guide.
Hop, hop—balanced views abide!

🚥 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 clearly describes the missing video player icons at 300px width, which matches the primary change.
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-118299-block-fix-reopened

---

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

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}