Skip to content

Removing Loading Text#1472

Open
steam-bell-92 wants to merge 2 commits intorecodehive:mainfrom
steam-bell-92:Temp
Open

Removing Loading Text#1472
steam-bell-92 wants to merge 2 commits intorecodehive:mainfrom
steam-bell-92:Temp

Conversation

@steam-bell-92
Copy link
Copy Markdown
Member

Description

This PR removes loading text on give away page

Fixes #1466

Type of Change

  • New feature (e.g., new page, component, or functionality)
  • Bug fix (non-breaking change that fixes an issue)
  • UI/UX improvement (design, layout, or styling updates)
  • Performance optimization (e.g., code splitting, caching)
  • Documentation update (README, contribution guidelines, etc.)
  • Other (please specify):

Changes Made

  • removes loading text on give away page

Checklist

  • My code follows the style guidelines of this project.
  • I have tested my changes across major browsers and devices
  • My changes do not generate new console warnings or errors .
  • I ran npm run build and attached screenshot(s) in this PR.
  • This is already assigned Issue to me, not an unassigned issue.

Copilot AI review requested due to automatic review settings May 9, 2026 17:34
@vercel
Copy link
Copy Markdown
Contributor

vercel Bot commented May 9, 2026

@steam-bell-92 is attempting to deploy a commit to the recode Team on Vercel.

A member of the Team first needs to authorize it.

@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 9, 2026

Thank you for submitting your pull request! 🙌 We'll review it as soon as possible. The estimated time for response is 5–8 hrs.

In the meantime, please provide all necessary screenshots and make sure you run - npm build run , command and provide a screenshot, a video recording, or an image of the update you made below, which helps speed up the review and assignment. If you have questions, reach out to LinkedIn. Your contributions are highly appreciated!😊

Note: I maintain the repo issue every day twice at 8:00 AM IST and 9:00 PM IST. If your PR goes stale for more than one day, you can tag and comment on this same issue by tagging @sanjay-kv.

We are here to help you on this journey of open source. Consistent 20 contributions are eligible for sponsorship 💰

🎁 check our list of amazing people we sponsored so far: GitHub Sponsorship. ✨

📚Your perks for contribution to this community 👇🏻

  1. Get free Consultation use code recode50 to get free: Mentorship for free.

  2. Get the Ebook for free use code recode at checkout: Data Science cheatsheet for Beginners.

  3. Check out this weekly Newsletter: Sanjay's Newsletter.

If there are any specific instructions or feedback regarding your PR, we'll provide them here. Thanks again for your contribution! 😊

@github-actions github-actions Bot added in-review The current changes are in review and would need approval and testing before merging level 1 10 points recode this is label for leaderboard labels May 9, 2026
@github-actions github-actions Bot added this to the recode:launch 3.0 milestone May 9, 2026
@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 9, 2026

✅ Synchronized metadata from Issue #1466:

  • Labels: level 1, recode
  • Milestone: recode:launch 3.0

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR aims to remove the visible “loading” text/indicator shown on the dashboard Giveaway page (per issue #1466).

Changes:

  • Removed the Loading... text element from the leaderboard loading UI.
  • Removed several Framer Motion enter/in-view animations from the Giveaway page sections/cards.
  • Simplified SlotCounter usage in the stats card.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment on lines 764 to 767
{loading ? (
<div className="giveaway-loading">
<div className="loading-spinner">Loading...</div>
<p>Fetching leaderboard data...</p>
</div>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

in-review The current changes are in review and would need approval and testing before merging level 1 10 points recode this is label for leaderboard

Projects

None yet

Development

Successfully merging this pull request may close these issues.

🐞[Bug]: Irrelevant loading text on Giveaway page

3 participants