Skip to content

chore: webinar updates#189

Merged
hmbanan666 merged 2 commits into
mainfrom
webinar-chore
Sep 26, 2025
Merged

chore: webinar updates#189
hmbanan666 merged 2 commits into
mainfrom
webinar-chore

Conversation

@hmbanan666
Copy link
Copy Markdown
Collaborator

@hmbanan666 hmbanan666 commented Sep 26, 2025

Summary by CodeRabbit

  • UI/Style

    • Reduced header top margin on medium screens for tighter spacing.
    • Redesigned hero video placeholder with a background image, updated banner styling, and smaller icon/text; button label shortened.
    • Introduced a rotated yellow band wrapper for the scrolling titles section.
    • Logo now allows shrinking for improved responsiveness.
  • Removed

    • Color mode toggle button removed from social buttons; only Telegram and VK remain.
  • Configuration

    • Disabled UI color mode and removed the global color mode fallback, aligning the app to a single theme.

@hmbanan666 hmbanan666 self-assigned this Sep 26, 2025
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Sep 26, 2025

Caution

Review failed

The pull request is closed.

Walkthrough

UI-only adjustments across webinar components: reduced header margin, redesigned Hero video placeholder visuals, updated infinite titles divider wrapper, removed color mode button and wrapper from social buttons, and tweaked logo sizing class. Nuxt config disables UI color mode and removes the top-level colorMode block. No logic or control flow changes.

Changes

Cohort / File(s) Summary of changes
Header & Logo layout tweaks
apps/webinar/app/components/Header.vue, apps/webinar/app/components/Logo.vue
Header: decreased md breakpoint top margin (md:mt-10md:mt-6). Logo: removed shrink-0 from SVG class.
Hero video placeholder redesign
apps/webinar/app/components/HeroVideo.vue
Reworked placeholder when isVideoShown is false: new padding, background color/image, alignment, smaller icon/text, updated banner styling and text. Click-to-toggle behavior unchanged.
Carousel wrapper restructure
apps/webinar/app/components/InfiniteTitlesDivider.vue
Introduced full-width, rotated background wrapper around existing UCarousel; carousel config and item logic unchanged.
Social buttons & color mode removal
apps/webinar/app/components/SocialButtons.vue, apps/webinar/nuxt.config.ts
SocialButtons: removed outer wrapper and UColorModeButton; kept Telegram and VK UButtons. Nuxt: set ui.colorMode to false; removed top-level colorMode block.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

  • feat: webinar app #181 — Prior UI/config tweaks to the same webinar components and nuxt.config.ts.
  • fix: colors #96 — Adjusts Nuxt color mode settings similar to this PR (ui.colorMode and top-level colorMode).

Poem

I nudge the margins, light and neat,
A video peeks with fresher beat 🎬
The titles tilt, a sunny stripe,
Two buttons wave—less themey hype.
Configs hush the mode of night—
I thump my paw: looks just right! 🐇✨

✨ Finishing touches
  • 📝 Generate Docstrings
🧪 Generate unit tests
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch webinar-chore

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 030cccc and d6f8d44.

⛔ Files ignored due to path filters (1)
  • apps/webinar/public/video-preview.jpg is excluded by !**/*.jpg
📒 Files selected for processing (6)
  • apps/webinar/app/components/Header.vue (1 hunks)
  • apps/webinar/app/components/HeroVideo.vue (1 hunks)
  • apps/webinar/app/components/InfiniteTitlesDivider.vue (1 hunks)
  • apps/webinar/app/components/Logo.vue (1 hunks)
  • apps/webinar/app/components/SocialButtons.vue (1 hunks)
  • apps/webinar/nuxt.config.ts (1 hunks)

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.

@hmbanan666 hmbanan666 changed the title Webinar chore chore: webinar updates Sep 26, 2025
@sonarqubecloud
Copy link
Copy Markdown

@hmbanan666 hmbanan666 merged commit 05f47d7 into main Sep 26, 2025
7 of 8 checks passed
@hmbanan666 hmbanan666 deleted the webinar-chore branch September 26, 2025 11:15
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