Bashamega / TrendTrack

TrendTrack is a platform that keeps you updated on the latest trending repositories across GitHub.
https://trendtrack.vercel.app
MIT License
18 stars 14 forks source link

Feature/ux enhancements - Resolved #51 #54

Closed clydehenry3 closed 1 month ago

clydehenry3 commented 1 month ago

Description

Summary of Changes:

This update introduces a new hero section with custom content and ux/design improvements. The navigation bar has been updated to include the logo, and a custom hero image has been added to enhance the visual appeal of the landing page.

Related Issue:

This change addresses [Issue #51 ], which requested the addition of a hero section and logo placement in the navigation bar. Please see Line 26 in src/components/Hero.tsx to link sections together. Attempted to add a smooth scroll but it kept breaking the file.

Motivation and Context:

The purpose of these updates is to improve the user interface by making the landing page more visually engaging and providing clear branding through the logo and custom imagery in the hero section.

Dependencies:

No new dependencies were added as part of this update. Tailwind CSS was used to style the components.

Fixes #(issue)

Type of change

Checklist:

Screenshots (if applicable):

TrendTrack After
vercel[bot] commented 1 month ago

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
trendtrack ✅ Ready (Inspect) Visit Preview 💬 Add feedback Oct 4, 2024 8:03pm
clydehenry3 commented 1 month ago

@Bashamega Testing worked great on my machine. Is there anything within my code to improve it? Thanks.

Bashamega commented 1 month ago

@Bashamega Testing worked great on my machine. Is there anything within my code to improve it? Thanks.

The error is with linter:

22:73  Error: `'` can be escaped with `'`, `‘`, `'`, `’`.  react/no-unescaped-entities
Bashamega commented 1 month ago

Also please fix the responsivness

clydehenry3 commented 1 month ago

Also please fix the responsivness

I believe the responsive issue is within the cards or the viewport themselves. When using the dev tools and making the screen smaller, it responds great. However when using just the browser, it appears to be within the cards being too wide.

clydehenry3 commented 1 month ago

I have made the changes of spelling, symbols, and logo.

clydehenry3 commented 1 month ago

I'm also going to upload the hero image because when looking on my phone, it looks blurry even though it's an SVG. Going to see how I can improve this.