Skip to content

chore: webinar updates#187

Merged
hmbanan666 merged 1 commit into
mainfrom
webinar-chore
Sep 26, 2025
Merged

chore: webinar updates#187
hmbanan666 merged 1 commit into
mainfrom
webinar-chore

Conversation

@hmbanan666
Copy link
Copy Markdown
Collaborator

@hmbanan666 hmbanan666 commented Sep 26, 2025

Summary by CodeRabbit

  • Style
    • Updated the sitewide logo to a sharper, scalable SVG in the header and footer for improved clarity on all displays.
    • Added subtle motion effects: speaker cards now slide up on entry, and select homepage sections slide in from the left/right for a more engaging feel.
  • Refactor
    • Retired the legacy logo implementation to standardize on the new logo and simplify maintenance.

@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

Replaced HeaderLogo with a new Logo component in Header and Footer. Added the new Logo.vue component with an inline SVG. Removed the template content from HeaderLogo.vue. Added motion-preset utility classes to SpeakerCard and sections on the index page to apply entry animations.

Changes

Cohort / File(s) Summary
Logo usage swap (Header & Footer)
apps/webinar/app/components/Header.vue, apps/webinar/app/components/Footer.vue
Replaced HeaderLogo references with Logo in templates; structure and behavior unchanged.
Logo component adjustments
apps/webinar/app/components/Logo.vue, apps/webinar/app/components/HeaderLogo.vue
Added new Logo.vue rendering an inline SVG. Removed rendering template from HeaderLogo.vue.
Animation class additions
apps/webinar/app/components/SpeakerCard.vue, apps/webinar/app/pages/index.vue
Added motion-preset classes (motion-preset-slide-up, ...-slide-right, ...-slide-left) to containers for entry animations; no logic changes.

Sequence Diagram(s)

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

  • feat: webinar app #181 — Introduces/uses HeaderLogo in the webinar app; related as this PR replaces HeaderLogo with the new Logo and updates Header/Footer accordingly.

Poem

I nibble on code like clover so green,
Swapping a logo for SVG sheen.
Headers and footers now proudly glow,
With slidey bits joining the show.
Thump-thump—deploy! My paws approve,
A hop, a skip, a smooth new move. 🐇✨

✨ Finishing touches
🧪 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 56541c5 and 71e7b82.

📒 Files selected for processing (6)
  • apps/webinar/app/components/Footer.vue (1 hunks)
  • apps/webinar/app/components/Header.vue (1 hunks)
  • apps/webinar/app/components/HeaderLogo.vue (0 hunks)
  • apps/webinar/app/components/Logo.vue (1 hunks)
  • apps/webinar/app/components/SpeakerCard.vue (1 hunks)
  • apps/webinar/app/pages/index.vue (2 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 merged commit 5b45158 into main Sep 26, 2025
5 of 6 checks passed
@hmbanan666 hmbanan666 deleted the webinar-chore branch September 26, 2025 09:47
@sonarqubecloud
Copy link
Copy Markdown

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