-
Notifications
You must be signed in to change notification settings - Fork 144
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Beautify the cryptocurrency page #305
Comments
I would like to work on this issue under gssoc'23 @royalpinto007 |
Hi @karthiknadar1204, |
Perfect, Thank you |
@royalpinto007 Is it fine if i use the coingecko API to fetch data, it is much more diverse in the contents it has to offer, will try the existing API as well before arriving at an conclusion |
No issues! |
Have started the work, will try to share the preview of the page by tomorrow |
Yeah man, no worries! |
@royalpinto007 I have fetched the coin details onto the page using the coingecko API, as a temporary measure can I use vanilla CSS for styling the cards, I believe the project has been prominently using tailwind classes for styling so thought should inform you once, fine? |
@royalpinto007 I am done with the issue, have to make some responsiveness, can you have someone else add the responsive behaviour in a separate issue, I have created a basic skeleton of the contents from the API that includes the graohs and a lot more and have added some primary styling but having some clashes. |
I'll share the preview in a bit |
In my proposed solution, Since the data about the coins is being fetched via an API, we could add individual cards for each coin , each card will contain the logo of the coin, a name below the logo and we seem to be presenting 7 different details for each coin, we could place these seven details beautifully in the cards along with pagination making only up to 25 to 30 cards visible on one page.
We could Add hover effects to the cards along with box shadow and a view button so if anyone clicks on the cards, the graph of the coin is fetched and the details are now visible on a much larger card about the width of the screen which includes the graphs of the coin and all the details we were previously fetching from the API.
We could make the TradingView snaps carousel dynamic , like the carousel will move automatically to the right and short span of time, simultaneously the user will also have the option to tweak the carousel
The text was updated successfully, but these errors were encountered: