Skip to content

feat: updated partners section with correct logos#125

Merged
danielhe4rt merged 2 commits into
3.xfrom
feat/partners-definition
Nov 25, 2025
Merged

feat: updated partners section with correct logos#125
danielhe4rt merged 2 commits into
3.xfrom
feat/partners-definition

Conversation

@DiogoKaster
Copy link
Copy Markdown
Contributor

@DiogoKaster DiogoKaster commented Nov 25, 2025

Summary by CodeRabbit

  • New Features

    • Interactive hover effects added to partner logos with smooth transitions
    • Expanded partner list in events section
  • UI/Style Updates

    • Improved partner carousel layout and spacing for consistent logo sizing
    • Footer now uses the default logo sizing (previous size attribute removed)
    • Continuous scroll animation no longer pauses on hover

✏️ Tip: You can customize this high-level summary in your review settings.

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Nov 25, 2025

Walkthrough

Adds top margin and refactors partners carousel into consistent flex items with grayscale hover effects, swaps image paths to Laravel assets and increases entries; removes logoSize="sm" from the footer invocation; deletes the hover-pause rule so infinite-scroll animation no longer pauses on hover.

Changes

Cohort / File(s) Summary
Partners carousel view
app-modules/events/resources/views/components/themes/3pontos/components/sections/partners.blade.php
Added mt-8; replaced fixed-width list items with flex items (flex h-24 w-[300px] items-center justify-center); standardized <img> to h-full w-full object-contain; switched src to asset() paths and updated alt text; added cursor-pointer, grayscale by default, transition, and hover to remove grayscale; added more partner entries.
Homepage footer usage
app-modules/events/resources/views/components/themes/3pontos/homepage.blade.php
Removed logoSize="sm" attribute from x-he4rt::partials.footer component invocation.
CSS animation
app-modules/he4rt/resources/css/index.css
Removed hover rule that paused .animate-infinite-scroll animation so the animation runs continuously, including on hover.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

  • Pay attention to the partners Blade view for correct asset() paths and alt texts.
  • Verify visual/spacing effects (mt-8, flex sizing) in the partners carousel and that new entries render as expected.
  • Confirm the CSS change doesn't unintentionally affect other infinite-scroll uses.

Possibly related PRs

  • feat: event page #108 — Modifies the same partners.blade.php view (markup, classes, assets), likely coordinating partner-carousel changes.

Poem

🐰 I hopped through logos, gray then bright,
Flexed each row to set things right,
Assets nested, margins set,
Animations now won't fret,
A tiny rabbit cheers the sight.

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately reflects the main objective of the PR: updating the partners section with correct logos, which is the primary focus of the changeset.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch feat/partners-definition

📜 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 f59cf9a and 43c48fc.

📒 Files selected for processing (1)
  • app-modules/he4rt/resources/css/index.css (0 hunks)
💤 Files with no reviewable changes (1)
  • app-modules/he4rt/resources/css/index.css
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)
  • GitHub Check: Perform Pest Tests / Run

Comment @coderabbitai help to get the list of available commands and usage tips.

@danielhe4rt danielhe4rt merged commit 83c55a4 into 3.x Nov 25, 2025
6 checks passed
@danielhe4rt danielhe4rt deleted the feat/partners-definition branch November 25, 2025 21:37
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