Skip to content

"renders testimonials list, including names, roles, messages and ratings"#3903

Open
abhilashgedela28-lang wants to merge 1 commit into
JhaSourav07:mainfrom
abhilashgedela28-lang:hover
Open

"renders testimonials list, including names, roles, messages and ratings"#3903
abhilashgedela28-lang wants to merge 1 commit into
JhaSourav07:mainfrom
abhilashgedela28-lang:hover

Conversation

@abhilashgedela28-lang
Copy link
Copy Markdown
Contributor

Description

Fixes the isolated unit and integration testing requirements for WallOfLove component's interactive tooltips, hover states, touch events, and coordinates computation.

Fixes #2777

Implemented:

  • components/WallOfLove.tsx: Interactive grid component for testimonials with custom mouseenter, mousemove, mouseleave, and touchstart/touchend event handlers to display responsive tooltips at computed container-relative coordinates.
  • components/WallOfLove.mouse-interactivity.test.tsx: Five test cases validating mouse interaction, computed coordinates, touch gesture propagation, cursor style classes, and tooltip visibility transitions with 100% line and function coverage.

Pillar

  • 🎨 Pillar 1 — New Theme Design
  • 📐 Pillar 2 — Geometric SVG Improvement
  • 🕐 Pillar 3 — Timezone Logic Optimization
  • 🛠️ Other (Bug fix, refactoring, docs, testing)

Visual Preview

N/A (Unit/Integration Test Coverage additions)

Checklist before requesting a review:

  • I have read the CONTRIBUTING.md file.
  • I have tested these changes locally (localhost:3000/api/streak?user=YOUR_USERNAME).
  • I have run npm run format and npm run lint locally and resolved all errors (CI will fail otherwise).
  • My commits follow the Conventional Commits format (e.g., feat(themes): ..., fix(calculate): ...).
  • I have updated README.md if I added a new theme or URL parameter.
  • I have starred the repo.
  • I have made sure that i have only one commit to merge in this PR.
  • The SVG output matches the CommitPulse "premium quality" aesthetic standard (no raw elements, smooth animations, correct fonts).
  • (Recommended) I joined the CommitPulse Discord community for contributor discussions, mentorship, and faster PR support.

@vercel
Copy link
Copy Markdown
Contributor

vercel Bot commented Jun 5, 2026

@abhilashgedela28-lang is attempting to deploy a commit to the jhasourav07's projects Team on Vercel.

A member of the Team first needs to authorize it.

@abhilashgedela28-lang
Copy link
Copy Markdown
Contributor Author

can you explain where the conflicts are occurring and if you give me the issues I can resolve it and push it again

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Jun 5, 2026

⚠️ Hey @abhilashgedela28-lang, this PR has merge conflicts with the main branch.

Please pull the latest changes and resolve the conflicts so we can review it!

git fetch origin
git rebase origin/main
# resolve any conflicts, then:
git push --force-with-lease

Once resolved, the needs-rebase label will be removed automatically on the next check. 🙌

@github-actions github-actions Bot added the needs-rebase This PR has merge conflicts and needs a rebase. label Jun 5, 2026
Copy link
Copy Markdown
Collaborator

@Aamod007 Aamod007 left a comment

Choose a reason for hiding this comment

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

Hey there! Thanks for your contribution! We noticed some unrelated files (package-lock.json, .eslintignore, or cross-branch files) got included in this PR. Could you please revert those changes and ensure only the files meant for this feature/fix are included? Let us know when it's updated!

@Aamod007 Aamod007 added GSSoC 2026 mentor:Aamod007 level:beginner Small changes Usually isolated fixes or simple UI/text updates. labels Jun 5, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

GSSoC 2026 level:beginner Small changes Usually isolated fixes or simple UI/text updates. mentor:Aamod007 needs-rebase This PR has merge conflicts and needs a rebase.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

test(WallOfLove-mouse-interactivity): verify Interactive Tooltips, Cursor Hovers & Touch Event Propagation (Variation 5)

2 participants