Skip to content

fix: add scroll animation, customer flow, and rename to Restroly #32#204

Closed
HarshParmar029 wants to merge 2 commits into
rdodiya:gssoc_developfrom
HarshParmar029:gssoc_develop
Closed

fix: add scroll animation, customer flow, and rename to Restroly #32#204
HarshParmar029 wants to merge 2 commits into
rdodiya:gssoc_developfrom
HarshParmar029:gssoc_develop

Conversation

@HarshParmar029
Copy link
Copy Markdown
Contributor

  • Description Box me ye exact copy-paste kar do:

Issue Link

Closes #32

Changes Made

  • Added clean scroll-triggered connecting lines/visual elements on the homepage.
  • Implemented the complete Customer Flow: Scan QR ➔ Select Menu ➔ Add to Cart ➔ Place Order ➔ Live Updates ➔ UPI Payment.
  • Corrected all hardcoded product names from "RestroHub" to "Restroly".

Type of Change

  • New feature
  • Bug fix

Testing Performed

Frontend Testing

  • Component renders correctly
  • Responsive on mobile/tablet/desktop
  • No console errors

Functionality Demo

  • Will provide the screen recording/video link shortly in the comments below.
Screen.Recording.restrohub.mp4

)

## Changes Made

- Added new Customer Flow section in "How It Works"
- Customer Journey: Scan QR → Browse Menu → Add to Cart → Place Order → Live Updates → UPI Payment
- Added smooth scroll animation with connecting line
- Fixed missing visual element at bottom on scroll
- Fully responsive

Fixes rdodiya#32
@rdodiya
Copy link
Copy Markdown
Owner

rdodiya commented May 23, 2026

Hi @HarshParmar029 ,
Please don't change the coloura of the customer as well as admin flow. Keep the colours same as website colour based on current theme. Animation should be performed on scroll top as well as on down (reverse).

Reference image for coloura

590569386-0cd2217e-0bb9-4e0a-a40c-8fc4fbb0f625

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 updates the public landing page “How It Works” section to include a dual flow (restaurant owners + customers) with scroll-triggered animations and a connecting line, and aligns the frontend lockfile dependencies with the intended router/tooling versions.

Changes:

  • Added a new scroll-animated “For Customers” flow (steps + connecting line) to the Landing page.
  • Restructured the “How It Works” section into separate “For Restaurant Owners” and “For Customers” flows.
  • Updated package-lock.json to use react-router-dom@6.x and refresh related dependency resolutions.

Reviewed changes

Copilot reviewed 1 out of 2 changed files in this pull request and generated 5 comments.

File Description
RestroHub-FrontEnd/src/pages/public/Landing.jsx Adds the dual-flow “How It Works” UI plus scroll-triggered animations and connecting line behavior.
RestroHub-FrontEnd/package-lock.json Updates resolved dependency graph (notably react-router-dom to v6) and tooling package resolutions.
Files not reviewed (1)
  • RestroHub-FrontEnd/package-lock.json: Language not supported

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

Comment on lines +84 to +88
// Animate the connecting line
const line = document.getElementById('customer-connect-line');
if (line) {
setTimeout(() => {
line.style.height = '100%';
Comment on lines +95 to +98
// Observe all scroll-animate elements
document.querySelectorAll('.scroll-animate').forEach((el) => {
observer.observe(el);
});
Comment on lines +336 to +338
{/* Custom Styles for Animation */}
<style dangerouslySetInnerHTML={{
__html: `
return () => obs.disconnect();
}, []);

// Scroll Animation for Customer Flow (user requested insertion near top useEffects)
Comment on lines 2149 to 2151
"engines": {
"node": "^14.18.0 || >=16.0.0"
"node": "^20.19.0 || >=22.12.0"
},
@rdodiya
Copy link
Copy Markdown
Owner

rdodiya commented May 24, 2026

Closing this PR as new raised for same #214

@rdodiya rdodiya closed this May 24, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug] Missing visual element at bottom of homepage on scroll

3 participants