Repository navigation
Install new npm - #70
Closed
Hossein-Kelisa wants to merge 110 commits into
Closed
Hossein-Kelisa wants to merge 110 commits into
Hossein-Kelisa wants to merge 110 commits into
Conversation
New branch
header change
add new feature to explain text
start to update Itemdetails page
update Authorization page mostly in styling
Contributor
There was a problem hiding this comment.
Pull request overview
This PR updates npm dependencies and introduces a broad client UI refactor (Home/Auth/Profile/Result layouts), along with associated CSS and component re-organization.
Changes:
- Bumps dependencies (e.g.,
dotenv,vite) and refreshespackage-lock.json. - Refactors client UI: new Home components, updated Header/Sidebar behavior, Profile tab layout, Result page layout, and substantial CSS restyling.
- Updates README demo link and trims README content.
Reviewed changes
Copilot reviewed 44 out of 56 changed files in this pull request and generated 12 comments.
Show a summary per file
| File | Description |
|---|---|
| server/package.json | Bumps dotenv dependency. |
| README.md | Updates project title/structure and demo link text. |
| package-lock.json | Lockfile refresh reflecting dependency updates. |
| client/package.json | Bumps vite version. |
| client/src/pages/ResultPage.jsx | Restructures ResultPage layout (sidebar + view toggle grouping). |
| client/src/pages/ResultPage.css | New layout styling + responsive adjustments for ResultPage. |
| client/src/pages/ProfilePage.jsx | Reworks Profile page into tabbed UI and adds logout button. |
| client/src/pages/ProfilePage.css | Adds new tabbed profile styling and responsive rules. |
| client/src/pages/ItemDetailsPage.css | Visual tweaks to item details layout (borders/margins/radii). |
| client/src/pages/Home/Home.jsx | Switches Home to newly organized HomeComponents and adds sections. |
| client/src/pages/AuthPage.jsx | Updates auth page to new AuthComponents structure and background styling. |
| client/src/pages/AuthPage.css | Adds new auth-page background/overlay + glassmorphism container styles. |
| client/src/index.css | Adjusts global body padding/background/line-height. |
| client/src/components/ViewToggle.jsx | Reorders controls and updates class names for styling. |
| client/src/components/ViewToggle.css | Updates ViewToggle layout (column layout, responsive styling). |
| client/src/components/Sidebar.jsx | Adds overlay + outside click handling comments/structure adjustments. |
| client/src/components/Sidebar.css | Redesigned sidebar + overlay styling and responsive behavior. |
| client/src/components/SearchBar.css | Restyles search bar with responsive sizing. |
| client/src/components/Pagination.css | Adds small-screen pagination button sizing. |
| client/src/components/ItemCard.jsx | Restructures markup (container/content wrappers) and renames root class. |
| client/src/components/ItemCard.css | Updates card styling + list-view adjustments + responsive tweaks. |
| client/src/components/HomeComponents/SharePrompt.jsx | Updates heading casing and button class. |
| client/src/components/HomeComponents/SharePrompt.css | New SharePrompt styling (replacing removed old CSS). |
| client/src/components/HomeComponents/Shape.jsx | Fixes asset import path for shape.svg. |
| client/src/components/HomeComponents/Shape.css | Minor margin adjustments. |
| client/src/components/HomeComponents/ItemSlider.jsx | Adjusts imports to match new folder structure. |
| client/src/components/HomeComponents/ItemSlider.css | New ItemSlider styling under HomeComponents. |
| client/src/components/HomeComponents/HorizontalScrolling.jsx | New scrolling text component. |
| client/src/components/HomeComponents/HorizontalScrolling.css | Styling/animation for scrolling text. |
| client/src/components/HomeComponents/ExplainText.jsx | New “Welcome” explain text component. |
| client/src/components/HomeComponents/ExplainText.css | Updates styling for explain text component. |
| client/src/components/HomeComponents/ExplainText2.jsx | New second explain text component. |
| client/src/components/HomeComponents/ExplainText2.css | Styling for ExplainText2. |
| client/src/components/HomeComponents/Banner.jsx | New banner component in HomeComponents. |
| client/src/components/HomeComponents/Banner.css | New banner styling using asset background. |
| client/src/components/Header.jsx | Simplifies header login/profile icon behavior and sidebar mounting. |
| client/src/components/Header.css | Header becomes fixed with updated layout/responsive rules. |
| client/src/components/Footer.jsx | Converts footer to semantic <footer> with external link. |
| client/src/components/Footer.css | Replaces old footer layout with new variables + responsive sizing. |
| client/src/components/FilterSidebar.jsx | Renames wrapper class to main-filter-sidebar. |
| client/src/components/FilterSidebar.css | Restyles filter sidebar (but includes some stale selectors). |
| client/src/components/AuthComponents/RegisterForm.jsx | New Register form component in AuthComponents. |
| client/src/components/AuthComponents/LoginForm.jsx | New Login form component in AuthComponents. |
| client/src/components/AuthComponents/ForgotPass.jsx | New Forgot password form component. |
| client/src/components/AuthComponents/AuthToggle.jsx | Minor cleanup to auth toggle component. |
| client/src/components/AuthComponents/LogoutButton.jsx | New Logout button with confirmation dialog. |
| client/src/components/AuthComponents/LogoutButton.css | Styling for new logout button. |
| client/src/components/SharePrompt.css | Removes old SharePrompt CSS (moved under HomeComponents). |
| client/src/components/LogoutButton.css | Removes old LogoutButton CSS (moved under AuthComponents). |
| client/src/components/ItemSlider.css | Removes old ItemSlider CSS (moved under HomeComponents). |
| client/src/components/Banner.css | Removes old Banner CSS (moved under HomeComponents). |
Comments suppressed due to low confidence (1)
client/src/pages/ResultPage.css:109
- Responsive styles here still reference
.filter-sidebar, butFilterSidebarnow renders withclassName="main-filter-sidebar", so these rules won't apply on small screens. Update the selector to the new class.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| return; | ||
| } | ||
| // Fetch user profile data if logged in | ||
| fetch("http://localhost:3000/api/auth/me", { |
Comment on lines
130
to
134
| @@ -111,25 +134,17 @@ const ProfilePage = () => { | |||
| /> | |||
Comment on lines
150
to
154
| @@ -139,11 +154,11 @@ const ProfilePage = () => { | |||
| /> | |||
Comment on lines
42
to
44
| const handleItemClick = (id) => { | ||
| navigate(`/items/${id}`); | ||
| }; |
Comment on lines
111
to
113
| .filter-sidebar > div > button.filter-apply-btn { | ||
| margin-top: 1rem; | ||
| } |
Comment on lines
+1
to
+16
| .auth-page { | ||
| min-height: 100vh; | ||
| display: flex; | ||
| flex-direction: column; | ||
| background-size: cover; | ||
| background-position: center; | ||
| background-attachment: fixed; | ||
| font-family: "PT Serif", serif; | ||
| } | ||
| .auth-page::before { | ||
| content: ""; | ||
| position: absolute; | ||
| inset: 0; | ||
| background: rgba(0, 0, 0, 0.5); | ||
| z-index: 0; | ||
| } |
Comment on lines
18
to
+34
| .auth-container { | ||
| max-width: 400px; | ||
| width: 90%; | ||
| min-height: 350px; | ||
| margin: 60px auto; | ||
| margin: 12rem auto; | ||
| padding: 30px 20px; | ||
| background-color: #ffffff; | ||
| border-radius: 12px; | ||
| box-shadow: 0 4px 20px rgba(0, 0, 0, 0.2); | ||
| font-family: "PT Serif", serif; | ||
| transition: all 0.3s ease; | ||
| background: rgba(197, 197, 197, 0.15); | ||
| border-radius: 16px; | ||
| box-shadow: 0 8px 32px rgba(0, 0, 0, 0.3); | ||
| backdrop-filter: blur(12px); | ||
| -webkit-backdrop-filter: blur(12px); | ||
| border: 1px solid rgb(197, 207, 9); | ||
| color: white; | ||
| transition: | ||
| transform 0.3s ease, | ||
| box-shadow 0.3s ease; | ||
| } |
| - `mongodb-memory-server` || To mock out our database in our backend tests. See [docs](https://github.com/nodkz/mongodb-memory-server) | ||
| - `cors` || To open up our API. See [docs](https://github.com/expressjs/cors#readme) | ||
| - `mongoose` || To add schemas to our database. See [docs](https://mongoosejs.com/) | ||
| [Click here for the Demo version](https://sharewithus.onrender.com/) wait 60 sec |
| @@ -3,18 +3,18 @@ import axios from "axios"; | |||
| import { useNavigate } from "react-router-dom"; | |||
| const [userInfo, setUserInfo] = useState(null); | ||
| const [editing, setEditing] = useState(false); | ||
| const [activeTab, setActiveTab] = useState("profile"); | ||
| const navigate = useNavigate(); |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.