Skip to content
This repository has been archived by the owner on Nov 2, 2021. It is now read-only.

Re-designed home. Added cards. Please see attached screenshot #714

Closed
wants to merge 14 commits into from

Conversation

vivek-manjunath
Copy link

Redesigned home

@vercel
Copy link

vercel bot commented Apr 6, 2020

This pull request is being automatically deployed with ZEIT Now (learn more).
To see the status of your deployment, click below or on the icon next to each commit.

🔍 Inspect: https://zeit.co/covid19india/covid19india-react/2xp7jd14w
✅ Preview: https://covid19india-react-git-fork-vivek-manjunath-master.covid19india.now.sh

@limshark
Copy link

limshark commented Apr 6, 2020

UX wise , this is my suggestion

  1. First row title
  2. Second row Menu ( left side key menu items ) Right side FAQ
  3. Third row - Key Stats number as shown now
  4. Fourth row - Key Graphs
  5. fifth row - left half table , right half India's map.

@denvamsi
Copy link

denvamsi commented Apr 6, 2020

This looks very cool, But the header doesnt match. Can you try decreasing some font size and align menu items to center or float them left :)

@denvamsi
Copy link

denvamsi commented Apr 6, 2020

And the main thing is in mobile it isnt very responsive :)

@denvamsi
Copy link

denvamsi commented Apr 6, 2020

You can see below screenshot, The cards are going out of the box.

WhatsApp Image 2020-04-06 at 11 28 04 AM

@vivek-manjunath
Copy link
Author

Improved responsiveness. Please take a look
COVID-19 Tracker  India

@denvamsi
Copy link

denvamsi commented Apr 6, 2020

image
Is it looking good ? :)

@vivek-manjunath
Copy link
Author

Please have a look now.

image

@jeremyphilemon
Copy link
Collaborator

Hey thanks for the PR! Unfortunately, we'll not be moving forward with this redesign. However, thank you so much for your patience - feel free to open PRs against available issues :)

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants