Skip to content

Side pannel options#3418

Closed
ceoger wants to merge 7 commits intoOdyseeTeam:masterfrom
ceoger:side-pannel-options
Closed

Side pannel options#3418
ceoger wants to merge 7 commits intoOdyseeTeam:masterfrom
ceoger:side-pannel-options

Conversation

@ceoger
Copy link
Contributor

@ceoger ceoger commented Feb 11, 2026

Fixes

Issue Number:

What is the current behavior?

edit and download option does not show on shorts on web

What is the new behavior?

They all now show without having to scroll

Summary by CodeRabbit

Release Notes

  • Documentation

    • Updated help and support links across YouTube-related features to direct users to more specific guidance resources for channel claiming and syncing workflows.
  • Style

    • Adjusted YouTube Shorts side panel dimensions and positioning for improved layout consistency.
  • Content

    • Refined messaging in the YouTube channel linking flow with clearer, more positive phrasing for better user guidance.

@coderabbitai
Copy link

coderabbitai bot commented Feb 11, 2026

📝 Walkthrough

Walkthrough

Three UI files updated: YoutubeTransferStatus component and YoutubeSync page with revised help documentation links and user-facing copy; shorts stylesheet with adjusted side panel dimensions. No logic or control-flow modifications—changes are text, link destinations, and CSS spacing only.

Changes

Cohort / File(s) Summary
YouTube Transfer & Sync Documentation Links
ui/component/youtubeTransferStatus/view.jsx, ui/page/youtubeSync/view.jsx
Updated help link destinations across eligibility messages, channel linking, and self-sync features. Also revised user-facing copy for channel selection prompt and consent messaging to reflect positive, guided language.
Shorts Side Panel Styling
ui/scss/component/_shorts.scss
Expanded main side panel from 600px to 720px width with corresponding offscreen offset adjustment. Increased narrower breakpoint panel from 400px to 440px with updated positioning.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

Possibly related PRs

Poem

🐰 Links now point where guides reside,
Words reframed with gentler pride,
Panels stretched to wider sight,
Small tweaks make the UI bright! ✨

🚥 Pre-merge checks | ✅ 1 | ❌ 2
❌ Failed checks (2 warnings)
Check name Status Explanation Resolution
Title check ⚠️ Warning The title 'Side pannel options' is partially related to the changeset. It refers to the side panel width changes in the SCSS file, but does not capture the main changes: updated help/documentation links and improved YouTube sync UX copy. Revise the title to reflect the primary changes, such as 'Update YouTube sync links and improve sync UX copy' or 'Improve YouTube sync documentation links and channel setup guidance'.
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 (1 passed)
Check name Status Explanation
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

No actionable comments were generated in the recent review. 🎉


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.

}}
>
I want to sync my content to %site_name%. I have also read and understand %faq%.
I want to sync my content to %site_name%. I have read and agree to %terms%, and I understand
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

prob not worth it given the translations.

@tzarebczan
Copy link
Contributor

Try to only include 1 change in a PR. Did you mean for the other youtube sync changes to be here?

@ceoger
Copy link
Contributor Author

ceoger commented Feb 11, 2026

Was supposed be 2 different PRs, forgot to send the first one before the new changes.

Copy link
Contributor Author

@ceoger ceoger left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this is the only changes that's supposed to be in this pr but probably got mixed up with sync changes

@tzarebczan tzarebczan closed this Feb 12, 2026
@tzarebczan
Copy link
Contributor

please reopen a clean pr

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