Skip to content

Show file info without desc in shorts comment view#3415

Merged
tzarebczan merged 1 commit intomasterfrom
shorts-comments-more-info
Feb 11, 2026
Merged

Show file info without desc in shorts comment view#3415
tzarebczan merged 1 commit intomasterfrom
shorts-comments-more-info

Conversation

@keikari
Copy link
Contributor

@keikari keikari commented Feb 11, 2026

Fixes: If clicking shorts comment notification(or shorts comments in general), it's unclear what video you are on. Now shows the details also in comment view.

2026-02-11_15-37

Summary by CodeRabbit

  • Bug Fixes
    • Improved display logic to conditionally hide file descriptions in certain contexts, providing a cleaner user interface when appropriate.

@coderabbitai
Copy link

coderabbitai bot commented Feb 11, 2026

📝 Walkthrough

Walkthrough

The changes introduce an optional hideDescription prop to the FileTitleSection component that conditionally suppresses rendering of the FileDescription element. ShortsSidePanel utilizes this prop when rendering FileTitleSection in comments mode.

Changes

Cohort / File(s) Summary
FileTitleSection Enhancement
ui/component/fileTitleSection/view.jsx
Added optional hideDescription boolean prop to Props type and wired it to conditionally omit FileDescription rendering when true.
ShortsSidePanel Integration
ui/component/shortsSidePanel/view.jsx
Updated FileTitleSection render call to include hideDescription prop when isComments is true.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~5 minutes

Poem

A prop so small, yet oh so neat,
Hides descriptions when they should retreat,
Comments shine through, pristine and bright,
The FileTitle dances with delight! 🐰✨

🚥 Pre-merge checks | ✅ 2 | ❌ 1
❌ Failed checks (1 warning)
Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (2 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately describes the main change: adding the ability to show file information without the description in the shorts comment view, which aligns with the PR objective of making it clearer which video is being commented on.
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.

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

✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch shorts-comments-more-info

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.

@tzarebczan tzarebczan merged commit 651ca81 into master Feb 11, 2026
3 checks passed
@tzarebczan tzarebczan deleted the shorts-comments-more-info branch February 11, 2026 14:30
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